[MPlayer-dev-eng] [PATCH] fix numerous const issues in libmpcodecs

Rich Felker dalias at aerifal.cx
Fri Oct 19 14:44:48 CEST 2007


On Fri, Oct 19, 2007 at 08:20:27AM -0400, Alexander Strange wrote:
> 
> On Oct 19, 2007, at 4:48 AM, Reimar Döffinger wrote:
> 
> >> Index: libmpcodecs/vf_2xsai.c
> >> ===================================================================
> >> --- libmpcodecs/vf_2xsai.c	(revision 24811)
> >> +++ libmpcodecs/vf_2xsai.c	(working copy)
> >> @@ -16,6 +16,8 @@
> >>  #define uint16 unsigned short
> >>  #define uint8 unsigned char
> >>
> >> +/* FIXME: these all belong in the context, not as globals! */
> >> +
> >>  static uint32 colorMask = 0xF7DEF7DE;
> >>  static uint32 lowPixelMask = 0x08210821;
> >>  static uint32 qcolorMask = 0xE79CE79C;
> >
> > Just put that in an extra commit ;-)
> > As long as it compiles, I think it can't hurt to apply right away.
> >
> > Greetings,
> > Reimar Döffinger
> 
> -vf 2xsai only supports one colorspace anyway, so they should be made  
> const values.
> I remember this had a significant speedup when I did it ages ago to  
> the filters in SNES9X OS X.

There's code later in the filter that changes these variables... Is it
just cruft?

Rich



More information about the MPlayer-dev-eng mailing list