[FFmpeg-devel] [PATCH] doc/codecs: extend documentation for the threads option

Stefano Sabatini stefasab at gmail.com
Tue Sep 8 17:47:04 CEST 2015


On date Tuesday 2015-09-08 16:45:32 +0200, Clément Bœsch encoded:
> On Tue, Sep 08, 2015 at 03:59:42PM +0200, Stefano Sabatini wrote:
> > ---
> >  doc/codecs.texi | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> > 
> > diff --git a/doc/codecs.texi b/doc/codecs.texi
> > index da048b1..58f2f63 100644
> > --- a/doc/codecs.texi
> > +++ b/doc/codecs.texi
> > @@ -817,13 +817,17 @@ for codecs that support it. See also @file{doc/examples/export_mvs.c}.
> >  Deprecated, use mpegvideo private options instead.
> >  
> >  @item threads @var{integer} (@emph{decoding/encoding,video})
> > +Set the number of threads to be used, in case the selected codec
> > +implementation supports multi-threading.
> >  
> >  Possible values:
> >  @table @samp
> > - at item auto
> > -detect a good number of threads
> 
> > + at item auto (@emph{0})
> 
> "@item auto, 0" maybe?
> 
> > +select automatically the number of threads
> 
> "automatically select" sounds better to me but i'm not a native
> 
> >  @end table
> >  
> > +Default value is @samp{auto}.
> > +
> >  @item me_threshold @var{integer} (@emph{encoding,video})
> >  Set motion estimation threshold.
> 
> all my comments are nits, patch LGTM

Locally fixed, will push soon, thanks.
-- 
FFmpeg = Fanciful Free Magic Philosophical Elastic Gospel


More information about the ffmpeg-devel mailing list