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

[Paul B Mahol]

On 2/10/13, compn <tempn at twmi.rr.com> wrote:
> 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?

Please leave libswscale alone.

>
> -compn
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>