> 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.
signature.asc
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
