[MPlayer-dev-eng] [PATCH] (bit hackish) palette support for libav*

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Wed Jan 31 19:20:01 CET 2007


Hello,
On Mon, Jan 29, 2007 at 06:29:15PM +0100, Reimar D?ffinger wrote:
> On Mon, Jan 29, 2007 at 05:52:54PM +0100, Michael Niedermayer wrote:
> > On Mon, Jan 29, 2007 at 02:34:19PM +0100, Reimar Doeffinger wrote:
> > > attached patch fixes e.g.
> > > http://samples.mplayerhq.hu/V-codecs/KMVC/AIRSTRKE.AVI on big-endian
> > > and
> > > http://samples.mplayerhq.hu/game-formats/interplay-mve/baldursgate-camp.mve
> > > in general.
> > > Does it seems clean enough for you to be applied as-is (and yes, I
> > > noticed AVPaletteControl is deprecated and the later file probably
> > > should preferably be fixed in libavformat/libavcodec directly).
> > 
> > i refuse to accept any patch which depends on deprecated things
> > as this only increases the work for whoever removes AVPaletteControl
> 
> Forgot to say, attached is the bigendian palette fix, MPlayer expects it
> to be in the same order independent of endianness.
> Has nothing to do with AVPaletteControl ;-)
> I think it will have a problem with FF_BUFFER_HINTS_PRESERVE, I don't
> know the code well enough to know how to solve this.

Applied with a FIXME, it can't get any worse than it already is.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list