[MPlayer-dev-eng] [PATCH] Correctly support 32-bit raw MOV

Michael Niedermayer michaelni at gmx.at
Wed Jan 23 00:31:00 CET 2008


On Tue, Jan 22, 2008 at 02:51:42PM +0800, Timothy Lee wrote:
> Dear Carl,
> 
> Carl Eugen Hoyos wrote:
> > Hi!
> >
> > Timothy Lee<timothy.lee<at>  siriushk.com>  writes:
> >    
> >> The following patches fixes the support for 32-bit raw RGB video in
> >> MOV.  The colorspace of those video are IMGFMT_RGB32_1, which is not
> >> supported by libswscale (as mentioned in
> >> http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2007-
> >>      
> > Thank you for your work!
> >
> > Would it be possible to add a scaler to libswscale to support IMGFMT_RGB32_1? I
> > believe that solution would be preferred.
> >    
> I've tried down that path, but it appears to involve adding converter 
> functions for every output format, 

no, it would be possible to add just the bswap code and a 
rgb32_1 -> internal yuv converter
one could even add just the bswap32 filter and nothing else, that would be
ugly as many convertions would fail but it still is better than this hack


> so I opted for the filter approach 
> instead.

id say the bswap32 filter is rejected. ive no comment about the rest

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080123/6fde7922/attachment.pgp>


More information about the MPlayer-dev-eng mailing list