[FFmpeg-devel] [PATCH] libopenjpegdec: add pix fmt gbrp (9-14 bit)

[Paul B Mahol]

On 1/24/13, Michael Bradshaw <mjbshaw at gmail.com> wrote:
> On Wed, Jan 23, 2013 at 11:01 PM, Michael Cinquin
> <mc at michaelcinquin.com>wrote:
>
>> On 23 janv. 2013, at 22:05, Michael Bradshaw wrote:
>>
>> > On Wed, Jan 23, 2013 at 2:05 PM, Michael Bradshaw <mjbshaw at gmail.com>
>> wrote:
>> >
>> >> Added support for decoding gbrp (9-14 bit) images. I didn't add 8 and
>> >> 16
>> >> bit because these should just be decoded as RGB 8 and 16 bit.
>> >>
>> >> Note that this hasn't been tested. I don't have any samples, and I
>> >> can't
>> >> figure out how to get image_to_j2k to output 9-14 bit rgb samples (and
>> >> I
>> >> couldn't generate any with FFmpeg since there's no rgb48 -> gbr
>> converter).
>>
>> Here is a j2c in xyz colorspace :
>> www.michaelcinquin.com/get/12bpc_xyz.j2c
>> , and another j2c in rgb colorspace, both of them 12 bpc.
>> www.michaelcinquin.com/get/12bpc_rgb.j2c
>
>
> Thanks for the samples!
>
> To devs: this patch may not even be needed, actually (and after testing, it
> doesn't even work anyway). It looks like rgb48 pix fmt is selected and
> avctx->bits_per_raw_sample is properly set to 12 bpc (tested with the
> 12bpc_rgb.j2c sample).
>

You have no single clue what you are doing.

Did you ever notice that RGB48 case is pure hack
because you manully deinterleave/planarize RGB48.

Obviously you do not care for performance.

> Thanks,
>
> Michael
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>