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

Clément Bœsch u at pkh.me
Tue Sep 8 16:45:32 CEST 2015


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

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150908/4a3cb9ab/attachment.sig>


More information about the ffmpeg-devel mailing list