Re: [xine-devel] RGB frame support

[Michael R. <mr...@us...>]

Hi James,

> I've had this on the back-burner for a while now and thought I might
> as well get some feedback to see if people think it's a worthwhile
> thing to be doing...
>
> The attached patched adds an RGB32 frame type to xine, as well as
> support for this frame type to goom and XShm video out. Output is
> limited to unscaled 16- and 32-bit, but I think that adding the
> remaining conversion would be pretty easy (though I don't know how to
> do scaling). As you would expect the performance increase is quite
> significant.

Looks like really a great patch to me. Problems could arise in some 
video out plugins which up to now relied on only two frame types 
existing. They should be fixed and I would really like to see this 
patch in cvs, because support for RGB colourspace might be useful in 
the future for post plugins implementing video effects.

Michael

-- 
/* vsprintf.c -- Lars Wirzenius & Linus Torvalds. */
 * Wirzenius wrote this portably, Torvalds fucked it up :-)
 */
	2.2.16 /usr/src/linux/lib/vsprintf.c