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

Michael Niedermayer michael at niedermayer.cc
Thu Jan 14 13:06:49 CET 2016


On Thu, Jan 14, 2016 at 10:09:02AM +0000, Paul B Mahol wrote:
> On 1/14/16, Christophe Gisquet <christophe.gisquet at gmail.com> wrote:
> > Hi,
> >
> > 2016-01-14 9:41 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:
> >> 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 ?
> >
> > The required changes are probably non-trivial, but seeing how much
> > code that change actually impacted, I would feel very uncomfortable
> > next time someone else has to change this.
> >
> > I'm not sure about the road to follow. Surely there are people with
> > strong opinions on swscale that could provide some on this topic.
> 
> Leave it to the maintainer hands.

factored the define out and applied

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- 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/9e473a8d/attachment.sig>


More information about the ffmpeg-devel mailing list