[FFmpeg-devel] [PATCH] avutil/colorspace: Remove RGB_TO_Y/U/V

Clément Bœsch u at pkh.me
Sat Jun 6 15:05:59 CEST 2015


On Sat, Jun 06, 2015 at 02:57:15PM +0200, Michael Niedermayer wrote:
> They are unused and colorspace.h is not a installed header
> 
> Found-by: Timo Rothenpieler <timo at rothenpieler.org>
> ---
>  libavutil/colorspace.h |   12 ------------
>  1 file changed, 12 deletions(-)
> 
> diff --git a/libavutil/colorspace.h b/libavutil/colorspace.h
> index f438159..826ffd5 100644
> --- a/libavutil/colorspace.h
> +++ b/libavutil/colorspace.h
> @@ -84,18 +84,6 @@ static inline int C_JPEG_TO_CCIR(int y) {
>  }
>  
>  
> -#define RGB_TO_Y(r, g, b) \
> -((FIX(0.29900) * (r) + FIX(0.58700) * (g) + \
> -  FIX(0.11400) * (b) + ONE_HALF) >> SCALEBITS)
> -
> -#define RGB_TO_U(r1, g1, b1, shift)\
> -(((- FIX(0.16874) * r1 - FIX(0.33126) * g1 +         \
> -     FIX(0.50000) * b1 + (ONE_HALF << shift) - 1) >> (SCALEBITS + shift)) + 128)
> -
> -#define RGB_TO_V(r1, g1, b1, shift)\
> -(((FIX(0.50000) * r1 - FIX(0.41869) * g1 -           \
> -   FIX(0.08131) * b1 + (ONE_HALF << shift) - 1) >> (SCALEBITS + shift)) + 128)
> -
>  #define RGB_TO_Y_CCIR(r, g, b) \
>  ((FIX(0.29900*219.0/255.0) * (r) + FIX(0.58700*219.0/255.0) * (g) + \
>    FIX(0.11400*219.0/255.0) * (b) + (ONE_HALF + (16 << SCALEBITS))) >> SCALEBITS)

Fine with me

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150606/c4e34400/attachment.asc>


More information about the ffmpeg-devel mailing list