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

[Michael Niedermayer]

On Sat, Jun 06, 2015 at 03:05:59PM +0200, Clément Bœsch wrote:
> 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

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150606/4e4939d3/attachment.asc>