[FFmpeg-devel] [PATCH]

[Gyan]

On 15-05-2019 02:28 PM, Werner Robitza wrote:
> This fixes documentation for swscaler which is not reflecting what is
> shown when running ffmpeg -h full.
>
> Best regards,
> Werner

> From a460bae885499cb7782c096336cb351dfe6652ad Mon Sep 17 00:00:00 2001
> From: Werner Robitza <werner.robitza at gmail.com>
> Date: Wed, 15 May 2019 10:52:00 +0200
> Subject: [PATCH] doc/swscaler: fix documentation of pixel format and range
> 
> The possible values for source and destination pixel format, as well as the
> correct values for source and destination range are printed when running
> ffmpeg -h full, but are incorrect in the documentation. This patch fixes
> the documentation accordingly.

Which lines in the CLI help?

> Signed-off-by: Werner Robitza <werner.robitza at gmail.com>
> ---
>  doc/scaler.texi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/doc/scaler.texi b/doc/scaler.texi
> index f73804adfe..c873d6b2a7 100644
> --- a/doc/scaler.texi
> +++ b/doc/scaler.texi
> @@ -81,16 +81,16 @@ Set destination width.
>  Set destination height.
>  
>  @item src_format
> -Set source pixel format (must be expressed as an integer).
> +Set source pixel format.

I don't see any constants set in the AVOptions struct. Can you share a command line where you could set this option using a string?

>  @item dst_format
> -Set destination pixel format (must be expressed as an integer).
> +Set destination pixel format.

^ same as above.

>  @item src_range
> -Select source range.
> +If set to 1, source range will be full range.

Mention the range of values possible and their meaning.
  
>  @item dst_range
> -Select destination range.
> +If set to 1, destination range will be full range.

^ same as above.

>  @anchor{sws_params}
>  @item param0, param1
> -- 
> 2.21.0


Gyan