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

[Gyan Doshi]

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.

Regards,
Gyan