[MPlayer-dev-eng] [PATCH]Fix FFmpeg tiff RGBA on big endian
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Mon Jan 23 22:04:57 CET 2012
On Sun, Jan 22, 2012 at 11:42:10PM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
>
> > > Attached fixes playback of RGBA tiff on BE.
> > > Alternative might be to completely remove RGB32 and friends but this is the
> > > smaller patch...
> >
> > What do you mean by the "completely remove" part?
>
> Remove all RGB32 and BGR32 and replace with RGBA, ARGB, BGRA and ABGR.
> But I would prefer not to do that atm.
The idea is to have exactly those that FFmpeg actually supports.
If you removed RGB32 for example you'd have to add both RGBA and
ABGR (or was it ARGB and BGRA? Whatever) despite the fact that
only even one will ever work with a certain MPlayer build.
That seems to me like making stuff more complicated and increasing
the risk that it will only work on little-endian with not really
much of a benefit I can see.
More information about the MPlayer-dev-eng
mailing list