[FFmpeg-devel] [PATCH] lavc/libopenh264enc: apply minor consistency fixes to options text

Gregory J Wolfe gregory.wolfe at kodakalaris.com
Tue Sep 8 19:00:59 CEST 2015


> -----Original Message-----
> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces at ffmpeg.org] On Behalf
> Of Stefano Sabatini
> Sent: Tuesday, September 08, 2015 11:52 AM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH] lavc/libopenh264enc: apply minor
> consistency fixes to options text
> 
> On date Tuesday 2015-09-08 16:41:57 +0200, Clément Bœsch encoded:
> > On Tue, Sep 08, 2015 at 03:59:58PM +0200, Stefano Sabatini wrote:
> > > ---
> > >  libavcodec/libopenh264enc.c | 12 ++++++------
> > >  1 file changed, 6 insertions(+), 6 deletions(-)
> > >
> > > diff --git a/libavcodec/libopenh264enc.c
> > > b/libavcodec/libopenh264enc.c index b315e8b..84a9cb1 100644
> > > --- a/libavcodec/libopenh264enc.c
> > > +++ b/libavcodec/libopenh264enc.c
> > > @@ -46,12 +46,12 @@ typedef struct SVCContext {  #define OFFSET(x)
> > > offsetof(SVCContext, x)  #define VE AV_OPT_FLAG_VIDEO_PARAM |
> > > AV_OPT_FLAG_ENCODING_PARAM  static const AVOption options[] = {
> > > -    { "slice_mode", "Slice mode", OFFSET(slice_mode),
> AV_OPT_TYPE_INT, { .i64 = SM_AUTO_SLICE }, SM_SINGLE_SLICE,
> SM_RESERVED, VE, "slice_mode" },
> > > -    { "fixed", "A fixed number of slices", 0, AV_OPT_TYPE_CONST, { .i64 =
> SM_FIXEDSLCNUM_SLICE }, 0, 0, VE, "slice_mode" },
> > > -    { "rowmb", "One slice per row of macroblocks", 0,
> AV_OPT_TYPE_CONST, { .i64 = SM_ROWMB_SLICE }, 0, 0, VE, "slice_mode" },
> > > -    { "auto", "Automatic number of slices according to number of
> threads", 0, AV_OPT_TYPE_CONST, { .i64 = SM_AUTO_SLICE }, 0, 0, VE,
> "slice_mode" },
> > > -    { "loopfilter", "Enable loop filter", OFFSET(loopfilter),
> AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 1, VE },
> > > -    { "profile", "Set profile restrictions", OFFSET(profile),
> AV_OPT_TYPE_STRING, { 0 }, 0, 0, VE },
> >
> > > +    { "slice_mode", "set lice mode", OFFSET(slice_mode),

"set lice mode" probably should be "set slice mode".

> > > + AV_OPT_TYPE_INT, { .i64 = SM_AUTO_SLICE }, SM_SINGLE_SLICE,
> > > + SM_RESERVED, VE, "slice_mode" },
> >
> > slice
> 
> Locally fixed.
> 
> > > +        { "fixed", "a fixed number of slices", 0, AV_OPT_TYPE_CONST, { .i64
> = SM_FIXEDSLCNUM_SLICE }, 0, 0, VE, "slice_mode" },
> > > +        { "rowmb", "one slice per row of macroblocks", 0,
> AV_OPT_TYPE_CONST, { .i64 = SM_ROWMB_SLICE }, 0, 0, VE, "slice_mode" },
> > > +        { "auto", "automatic number of slices according to number of
> threads", 0, AV_OPT_TYPE_CONST, { .i64 = SM_AUTO_SLICE }, 0, 0, VE,
> "slice_mode" },
> > > +    { "loopfilter", "enable loop filter", OFFSET(loopfilter),
> AV_OPT_TYPE_INT, { .i64 = 1 }, 0, 1, VE },
> > > +    { "profile", "set profile restrictions", OFFSET(profile),
> > > + AV_OPT_TYPE_STRING, { 0 }, 0, 0, VE },
> > >      { NULL }
> > >  };
> >
> > rest LGTM
> 
> Will push soon, thanks.
> --
> FFmpeg = Fabulous and Friendly Merciful Powered Epic Geisha
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list