[FFmpeg-cvslog] r19548 - trunk/configure

Diego Biurrun diego
Thu Jul 30 23:28:50 CEST 2009


On Thu, Jul 30, 2009 at 09:45:01PM +0100, M?ns Rullg?rd wrote:
> diego <subversion at mplayerhq.hu> writes:
> >
> > Log:
> > Undefine av_always_inline before setting it to an empty value.
> > This avoids a ton of redefinition warnings.
> > patch by Jeff Downs, heydowns borg com
> >
> > --- trunk/configure	Thu Jul 30 21:14:57 2009	(r19547)
> > +++ trunk/configure	Thu Jul 30 22:28:07 2009	(r19548)
> > @@ -2655,6 +2655,7 @@ print_config CONFIG_ $TMPH config.mak $C
> >  echo "#define restrict $_restrict" >> $TMPH
> >
> >  if enabled small; then
> > +    echo "#undef av_always_inline"  >> $TMPH
> >      echo "#define av_always_inline"  >> $TMPH
> >  fi
> 
> Wrong fix.  The real issue is that config.h is included at the end of
> libavutil/common.h, after av_always_inline has already been defined.
> With this change, an av_always_inline definition in config.h will not
> affect any functions defined in common.h (not sure if there are any at
> the moment).  The correct fix is to include config.h near the top as
> any sane person would do it.

OK, I will commit this in a moment after a test compile.

Diego



More information about the ffmpeg-cvslog mailing list