On Thu, Jan 14, 2016 at 08:58:20AM +0100, Christophe Gisquet wrote: > > -#define YUVRGB_TABLE_HEADROOM 256 > > +#define YUVRGB_TABLE_HEADROOM 512 > [...] > > - const int yoffs = fullRange ? 384 : 326; > > + const int yoffs = fullRange ? 896 : 838; > > I think it's time to use that macro everywhere it is actually used without > showing up.
there are 2 seperate tables, the headroom in them also differs curently, also the exact relation between input values and headroom needed in the 2nd table depends on the yuv-rgb coefficients i can replace the 2nd tables numbers by a define if people want ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who are too smart to engage in politics are punished by being governed by those who are dumber. -- Plato
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
