[FFmpeg-devel] [PATCH] swscale/yuv2rgb: Increase YUV2RGB table headroom

Michael Niedermayer michael at niedermayer.cc
Thu Jan 14 09:41:43 CET 2016


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 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160114/d2744ab9/attachment.sig>


More information about the ffmpeg-devel mailing list