Jean-Francois, The two red tables actually do contain values greater then 127 so would not fit within a s8 variable, the other four i think may have only values between -128 and 127 but i haven't gone through and made sure of that. Brian On Fri, Sep 4, 2009 at 5:24 AM, Jean-Francois Moine<moinejf@xxxxxxx> wrote: > On Thu, 20 Aug 2009 19:08:58 -0700 > Joe Perches <joe@xxxxxxxxxxx> wrote: > >> Compiled, not tested, no hardware >> >> Reduces size of object >> >> Use s16 instead of int where possible. > [snip] >> -static const int hsv_red_x[] = { >> +static const s16 hsv_red_x[] = { >> 41, 44, 46, 48, 50, 52, 54, 56, >> 58, 60, 62, 64, 66, 68, 70, 72, >> 74, 76, 78, 80, 81, 83, 85, 87, > [snip] > > Hi Joe and Brian, > > I got the patch but I was wondering if such tables could be even > smaller with 's8'? > > Cheers. > > -- > Ken ar c'hentañ | ** Breizh ha Linux atav! ** > Jef | http://moinejf.free.fr/ > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html