[Ffmpeg-devel] [PATCH] move BE/LE macros to libavutil

Reimar Döffinger Reimar.Doeffinger
Sun Dec 3 17:45:49 CET 2006


Hello,
On Sun, Dec 03, 2006 at 05:08:07PM +0100, Michael Niedermayer wrote:
> On Sun, Dec 03, 2006 at 03:49:23PM +0100, Reimar D?ffinger wrote:
> > On Sun, Nov 19, 2006 at 05:38:50PM +0100, Michael Niedermayer wrote:
> > > On Sun, Nov 19, 2006 at 12:44:24PM +0100, Reimar D?ffinger wrote:
> > > > do you think this is okay, as in the attached patch? I already know a
> > > > few place in MPlayer that could avoid code duplication with that.
> > > > Though I also wonder if they should have a FF prefix (though they seem
> > > > to be constructed to allow them to be overridden by defines in system
> > > > headers?).
> > > > Any comments?
> > > 
> > > yes, there are read/write 16/32/64 little/big/native endian cases
> > > we have BE_* LE_* ST* LD* unaligned* and probably more
> > > IMHO we should first clean these up inside lavc before exporting them
> > 
> > Would maybe attached patch be acceptable? It does not really clean them
> > up, but neither does it export them. At least it collects them all in
> > one place.
> 
> ok

Good, applied.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list