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

[Michael Bradshaw]

On Thu, Jan 24, 2013 at 5:12 AM, Paul B Mahol <onemda at gmail.com> wrote:

> On 1/24/13, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> > Paul B Mahol <onemda <at> gmail.com> writes:
> >
> >> Did you ever notice that RGB48 case is pure hack
> >> because you manully deinterleave/planarize RGB48.
> >
> > (Apart from the fact that as you know there is no
> > complete planar GBR support making the discussion
> > not really helpful:)
> > Wasn't that implemented before non-8 bit GBR
> > code was added or do I miss something?
> >
> >> Obviously you do not care for performance.
> >
> > When I last tested libopenjpeg, it was a magnitude
> > slower than our native codec, so I don't understand
> > how a performance argument for this wrapper is
> > relevant.
>
> If time spent in wrapper can be made smaller
> it is better.
>
> Yes, GBRP case can take priority over RGB case once
> GBRP gets output in swscale. Currently it is still possible to output
> both via user request.


Yes, this works (without this patch, which is one reason why this patch is
so useless). Thanks for your time, Paul.