[MPlayer-cvslog] r26219 - trunk/mpbswap.h

Diego Biurrun diego at biurrun.de
Wed Mar 12 11:12:44 CET 2008


On Tue, Mar 11, 2008 at 07:30:20PM +0100, Aurelien Jacobs wrote:
> Ivan Kalvachev wrote:
> 
> > On Tue, Mar 11, 2008 at 1:29 PM, Reimar Döffinger
> > <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> > >
> > > On Tue, Mar 11, 2008 at 01:20:31PM +0200, Ivan Kalvachev wrote:
> > >  > On Tue, Mar 11, 2008 at 12:42 PM, Reimar Döffinger
> > >  > <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> > >  > > On Mon, Mar 10, 2008 at 09:20:08PM +0100, diego wrote:
> > >  > >  >
> > >  > >  > Log:
> > >  > >  > Add missing header #include.
> > >  > >  >
> > >  > >  > --- trunk/mpbswap.h   (original)
> > >  > >  > +++ trunk/mpbswap.h   Mon Mar 10 21:20:08 2008
> > >  > >  > @@ -3,6 +3,8 @@
> > >  > >  >
> > >  > >  >  #include <sys/types.h>
> > >  > >  >  #include "libavutil/bswap.h"
> > >  > >  > +#include "config.h"
> > >  > >
> > >  > >  Uh, libavutil/bswap.h almost certainly requires config.h,
> > >  > > this seems like a stupid place to put it?
> 
> Uh, libavutil/bswap.h almost certainly doesn't require anything as
> it is supposed to be self-contained.

It does, in fact, require config.h without #including it directly.  But
that is a bug in FFmpeg that I shall fix shortly.

Diego



More information about the MPlayer-cvslog mailing list