[FFmpeg-devel] [PATCH 1/4] Doxygen sws_getColorspaceDetails(), and use saner names for the used tables.

Stefano Sabatini stefano.sabatini-lala
Sun May 23 12:09:33 CEST 2010


On date Sunday 2010-05-23 12:06:42 +0200, Stefano Sabatini encoded:
> ---
>  swscale.h |   22 +++++++++++++++++-----
>  utils.c   |   18 +++++++++---------
>  2 files changed, 26 insertions(+), 14 deletions(-)
> 
> diff --git a/swscale.h b/swscale.h
> index f6e0ccf..888973c 100644
> --- a/swscale.h
> +++ b/swscale.h
> @@ -205,12 +205,24 @@ int sws_scale_ordered(struct SwsContext *context, const uint8_t* const src[],
>  #endif
>  
>  /**
> - * @param inv_table the yuv2rgb coefficients, normally ff_yuv2rgb_coeffs[x]
> - * @param fullRange if 1 then the luma range is 0..255 if 0 it is 16..235
> - * @return -1 if not supported

As for this I'm not sure amongst these:

1) keep the current documentation, and eventually implement a
sws_setColorspaceDetails2() returning a sane value

2) consider this like a documentation bug, fix it like in the patch,
and eventually bumping minor/micro.

Regards.
-- 
FFmpeg = Fostering & Faithful Mean Ponderous Elected Gem



More information about the ffmpeg-devel mailing list