On Fri, Jun 19, 2020 at 06:50:03PM +0200, Carl Eugen Hoyos wrote: > Am Sa., 30. Mai 2020 um 15:29 Uhr schrieb <[email protected]>: > > > > From: Limin Wang <[email protected]> > > > > Signed-off-by: Limin Wang <[email protected]> > > --- > > doc/encoders.texi | 14 ++++++++++++++ > > libavcodec/mpeg12enc.c | 2 ++ > > libavcodec/profiles.h | 8 ++++++++ > > 3 files changed, 24 insertions(+) > > > > diff --git a/doc/encoders.texi b/doc/encoders.texi > > index 954f0f6..1331b79 100644 > > --- a/doc/encoders.texi > > +++ b/doc/encoders.texi > > @@ -2740,6 +2740,20 @@ MPEG-2 video encoder. > > @subsection Options > > > > @table @option > > +@item profile @var{integer} > > +Select the mpeg2 profile to encode: > > + > > +@table @samp > > +@item 422 > > +@item main > > +@item ss > > +Spatially Scalable > > +@item snr > > +SNR Scalable > > +@item high > > +@item simple > > +@end table > > Above looks wrong (as reported on the user mailing list): > high and main have to be swapped, no?
User can't assume the integer value by the order, it's const string. I recall ffmpeg -h full can give the detail integer, now it's missing for the profile options, what's the problem? But I'll fix the order to avoid such confusing. > > Carl Eugen > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". -- Thanks, Limin Wang _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
