[FFmpeg-devel] the proper way to handle a special colorspace?

[compn]

On Sun, 10 Feb 2013 17:29:00 +0100, Michael Niedermayer wrote:
>On Sun, Feb 10, 2013 at 05:08:43PM +0100, u-bo1b at 0w.se wrote:
>> Hello,
>> 
>> I am trying to make the Cinepak decoder in FFmpeg compliant with
>> the de-facto Cinepak format definition, which uses a non-standard colorspace.
>> 
>> What would be the proper place and way to do the conversion?
>> 
>> The pixel format is declared as YUP420P which is almost true, but the
>> conversion matrix which for the format is defined from/to BGR24 must
>> be different.
>> 
>> Where in FFmpeg source should the fix belong to possibly reuse the
>> existing conversion code (which code?)?
>
>see swscale sws_setColorspaceDetails(), ff_yuv2rgb_coeffs and related
>code
>
>also see AVColorSpace and structs close by

i think hes asking if it should be a new colorspace in swscale, or
added into the cinepak decoder/encoder common code.

by your answer i think it has to be a new swscale colorspace?

-compn