[MPlayer-cvslog] r31015 - trunk/libvo/vo_corevideo.m

[Reimar Döffinger]

On Mon, Apr 05, 2010 at 03:14:00AM -0500, Samuel Benson wrote:
> 
> On Apr 5, 2010, at 2:48 AM, Reimar Döffinger wrote:
> 
> > On Mon, Apr 05, 2010 at 09:33:04AM +0200, Reimar Döffinger wrote:
> >> On Mon, Apr 05, 2010 at 09:22:21AM +0200, reimar wrote:
> >>> Author: reimar
> >>> Date: Mon Apr  5 09:22:21 2010
> >>> New Revision: 31015
> >>> 
> >>> Log:
> >>> Enable OSD also for BGR32, the code is the same as for RGB32.
> >> 
> >> Could someone please test with -vf format=rgb32 and -vf format=bgr32
> >> on an Intel MAC?
> > 
> > Btw. I think attached patch fixes it, though initialization will fail
> > for one of those above (which depends on endianness).
> > In addition, -vf format=rgb24 should work.
> 
> Patch does enable -vf format=bgr32 to work under corevideo, but causes rgb32 to
> exit with a cannot initialize filter.

That is the expected behaviour, rgb32 is unsupported by corevideo on little-endian.
Thanks for testing, since reading your mail it seems to work as expected on Intel
and I confirmed it on PPC, applied.