diff options
| author | Alan Curry <pacman@world.std.com> | 2006-02-09 00:29:35 +0000 |
|---|---|---|
| committer | Diego Biurrun <diego@biurrun.de> | 2006-02-09 00:29:35 +0000 |
| commit | 7d20ebffdf3313d2194452d685072ec580edfa16 (patch) | |
| tree | 336bc840537193c19c44883f0eb6d8f8d0cf37f1 /postproc | |
| parent | 9870a7bdb897fe66bfd029ff1ba90f9b3653ecf2 (diff) | |
| download | ffmpeg-7d20ebffdf3313d2194452d685072ec580edfa16.tar.gz | |
Correct RGB vs. BGR confusion, the macros vec_mstrgb24 and vec_mstbgr24 each
do the opposite of what they say.
The vec_mstrgb24 and vec_mstbgr24 macros should be opposites and not perform
the same things.
patch by Alan Curry, pacman_at_world_dot_std_dot_com
Originally committed as revision 17564 to svn://svn.mplayerhq.hu/mplayer/trunk/postproc
Diffstat (limited to 'postproc')
| -rw-r--r-- | postproc/yuv2rgb_altivec.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/postproc/yuv2rgb_altivec.c b/postproc/yuv2rgb_altivec.c index eea7134e96..2d2f7766b2 100644 --- a/postproc/yuv2rgb_altivec.c +++ b/postproc/yuv2rgb_altivec.c @@ -142,7 +142,7 @@ do { \ y2 = vec_perm (o3,o2,perm_rgb_3); \ } while(0) -#define vec_mstrgb24(x0,x1,x2,ptr) \ +#define vec_mstbgr24(x0,x1,x2,ptr) \ do { \ typeof(x0) _0,_1,_2; \ vec_merge3 (x0,x1,x2,_0,_1,_2); \ @@ -151,7 +151,7 @@ do { \ vec_st (_2, 0, ptr++); \ } while (0); -#define vec_mstbgr24(x0,x1,x2,ptr) \ +#define vec_mstrgb24(x0,x1,x2,ptr) \ do { \ typeof(x0) _0,_1,_2; \ vec_merge3 (x2,x1,x0,_0,_1,_2); \ @@ -417,7 +417,7 @@ static int altivec_##name (SwsContext *c, \ #define out_rgba(a,b,c,ptr) vec_mstrgb32(typeof(a),a,b,c,((typeof (a))AVV(0)),ptr) #define out_argb(a,b,c,ptr) vec_mstrgb32(typeof(a),((typeof (a))AVV(0)),a,b,c,ptr) #define out_rgb24(a,b,c,ptr) vec_mstrgb24(a,b,c,ptr) -#define out_bgr24(a,b,c,ptr) vec_mstbgr24(c,b,a,ptr) +#define out_bgr24(a,b,c,ptr) vec_mstbgr24(a,b,c,ptr) DEFCSP420_CVT (yuv2_abgr32, out_abgr) #if 1 |