[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