[FFmpeg-cvslog] r19548 - trunk/configure

Måns Rullgård mans
Thu Jul 30 22:45:01 CEST 2009


diego <subversion at mplayerhq.hu> writes:

> Author: diego
> Date: Thu Jul 30 22:28:07 2009
> New Revision: 19548
>
> 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
>
> Modified:
>    trunk/configure
>
> Modified: trunk/configure
> ==============================================================================
> --- 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.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-cvslog mailing list