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

[Stefano Sabatini]

On date Wednesday 2014-06-18 23:29:36 +0200, Michael Niedermayer encoded:
> On Mon, Jun 09, 2014 at 07:06:55PM +0200, Stefano Sabatini wrote:
> > Based on avcodec.h documentation.
> > ---
> >  doc/codecs.texi | 12 ++++++++++--
> >  1 file changed, 10 insertions(+), 2 deletions(-)
> > 
> > diff --git a/doc/codecs.texi b/doc/codecs.texi
> > index b5255cb..49c8669 100644
> > --- a/doc/codecs.texi
> > +++ b/doc/codecs.texi
> > @@ -1032,15 +1032,23 @@ Set the log level offset.
> >  Number of slices, used in parallelized encoding.
> >  
> >  @item thread_type @var{flags} (@emph{decoding/encoding,video})
> > -Select multithreading type.
> > +Select which multithreading methods to use.
> > +
> > +Use of @samp{frame} will increase decoding delay by one frame per
> > +thread, so clients which cannot provide future frames should not use
> > +it.
> >  
> >  Possible values:
> >  @table @samp
> >  @item slice
> > +Decode more than one part of a single frame at once.
> 

> iam not sure its mentioned elsewhere but this could also say something
> like
> "multithreading using slices works only when the video was encoded
>  with slices"

Good idea, added.

> 
> also, either way, patch LGTM

Will push it soon, thanks.
-- 
FFmpeg = Fabulous Foolish Majestic Puritan Exuberant Guru