[FFmpeg-devel] colorspace.h

[Timo Rothenpieler]

> The specific code I'm referring to is at the end of colorspace.h where it looks like they're trying to encode RGB to BT.601.
> 
> #define RGB_TO_Y(r1, g1, b1, shift)
> #define RGB_TO_U(r1, g1, b1, shift)
> #define RGB_TO_V(r1, g1, b1, shift)

A quick search over the ffmpeg codebase tells me that these functions
are not used anywhere, so if you are trying to fix some inaccuracy,
those macros are most certainly not where it comes from.

Also, keep in mind that those conversion routines look that weird
because they are optimized for speed.
Your implementation, while maybe more correct, might be several times
slower than the current code, which can be very significant, depending
on how they are used.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150606/6a833456/attachment.asc>