[FFmpeg-devel] [PATCH] doc/encoders/libxvid: fix references for me_quality option

Stefano Sabatini stefasab at gmail.com
Fri Oct 6 10:54:00 EEST 2023


On date Wednesday 2023-09-06 10:56:43 +0530, Gyan Doshi wrote:
> On 2023-08-26 04:26 pm, Stefano Sabatini wrote:
> > Drop reference to constants removed in 94eed68ace9f2416af8.
> > 
> > In particular, rename me_method to me_quality and add description for
> > supported values.
> > 
> > Address trac issue:
> > http://trac.ffmpeg.org/ticket/10003
> > ---
> >   doc/encoders.texi | 21 +++++++++------------
> >   1 file changed, 9 insertions(+), 12 deletions(-)
> > 
> > diff --git a/doc/encoders.texi b/doc/encoders.texi
> > index 14c69fc7b3..25c40a6486 100644
> > --- a/doc/encoders.texi
> > +++ b/doc/encoders.texi
> > @@ -2981,27 +2981,24 @@ Place global headers in extradata instead of every keyframe.
> >   @item trellis
> > - at item me_method
> > -Set motion estimation method. Possible values in decreasing order of
> > + at item me_quality
> > +Set motion estimation quality level. Possible values in decreasing order of
> >   speed and increasing order of quality:
> >   @table @samp
> > - at item zero
> > + at item 0
> >   Use no motion estimation (default).
> > - at item phods
> > - at item x1
> > - at item log
> > + at item 1, 2
> >   Enable advanced diamond zonal search for 16x16 blocks and half-pixel
> > -refinement for 16x16 blocks. @samp{x1} and @samp{log} are aliases for
> > - at samp{phods}.
> > +refinement for 16x16 blocks.
> > - at item epzs
> > + at item 3, 4
> >   Enable all of the things described above, plus advanced diamond zonal
> > -search for 8x8 blocks, half-pixel refinement for 8x8 blocks, and motion
> > -estimation on chroma planes.
> > +search for 8x8 blocks and half-pixel refinement for 8x8 blocks, also
> > +enable motion estimation on chrom planes for P and B-frames.
> s/chrom/chroma
> 
> > - at item full
> > + at item 5, 6
> >   Enable all of the things described above, plus extended 16x16 and 8x8
> >   blocks search.
> >   @end table
> 
> Else, LGTM.

Applied, thanks.


More information about the ffmpeg-devel mailing list