[FFmpeg-user] MPEG2 Profile Options Integer order wrong

[lance.lmwang at gmail.com]

On Fri, Jun 19, 2020 at 08:03:00PM +0200, Marton Balint wrote:
> 
> 
> On Fri, 19 Jun 2020, Carl Eugen Hoyos wrote:
> 
> > Am Mi., 17. Juni 2020 um 01:32 Uhr schrieb Bill Crockett <wsc4 at optusnet.com.au>:
> > > 
> > > https://ffmpeg.org/ffmpeg-all.html#mpeg2
> > > 
> > > Who maintains the documentation on the above and how can I contact to ask if it is wrong or correct?
> > > 
> > > 16.15 mpeg2
> > > 
> > > MPEG-2 video encoder.
> > > 
> > > 16.15.1 Options
> > > 
> > > profile integer
> > > 
> > >     Select the mpeg2 profile to encode:
> > > 
> > >     ‘422’
> > >     ‘main’
> > >     ‘ss’
> > > 
> > >         Spatially Scalable
> > >     ‘snr’
> > > 
> > >         SNR Scalable
> > >     ‘high’
> > >     ‘simple’
> > > 
> > > There are no numbers listed, but my guess is the order as is written and would be "main" = 1 and "high" = 4
> > > 
> > > This is wrong and "high should be 1 and "main" should be 4.
> > 
> > Indeed.
> > 
> > I will wait a few more days and fix it then unless somebody beats me.
> 
> You can fix it if you want, but I don't know why anybody assumes that the
> order of the options and their integer mapping actually matches? Nothing
> says it does, and I bet there are other options where it does not. I think
> what confused the user here is that the option is mentioned as an "integer"
> option when in fact it is a string option from user point of view. Maybe we
> shuld simply remove the <integer> part, so the user can use the string
> equivavalent as generally advised.

I'll submit a patch to fix the order.  Also the ffpeg -h full can't get the profile
option anyore or it'll show the value for every const string.

> 
> Regards,
> Marton
> _______________________________________________
> ffmpeg-user mailing list
> ffmpeg-user at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-user
> 
> To unsubscribe, visit link above, or email
> ffmpeg-user-request at ffmpeg.org with subject "unsubscribe".

-- 
Thanks,
Limin Wang