[FFmpeg-cvslog] r12484 - in trunk: configure libavutil/internal.h

[Ramiro Polla]

Ramiro Polla wrote:
> mru wrote:
>> Author: mru
>> Date: Tue Mar 18 03:01:20 2008
>> New Revision: 12484
>>
>> Log:
>> check for prefix on extern symbols in configure
>>
>>
>> Modified:
>>    trunk/configure
>>    trunk/libavutil/internal.h
>>
>> Modified: trunk/configure
>> ============================================================================== 
>>
>> --- trunk/configure    (original)
>> +++ trunk/configure    Tue Mar 18 03:01:20 2008
>> @@ -1491,6 +1491,12 @@ if test "$?" != 0; then
>>      die "C compiler test failed."
>>  fi
>>  
>> +check_cc <<EOF || die "Symbol mangling check failed."
>> +int ff_extern;
>> +EOF
>> +sym=$(nm -P $TMPO)
>> +extern_prefix=${sym%%ff_extern*}
>> +
>>  check_asm inline_asm '""'
>>  
>>  if enabled x86; then
>> @@ -2070,6 +2076,7 @@ enabled asmalign_pot &&
>>      printf '#define ASMALIGN(ZEROBITS) ".align " #ZEROBITS 
>> "\\n\\t"\n' >> $TMPH ||
>>      printf '#define ASMALIGN(ZEROBITS) ".align 1<<" #ZEROBITS 
>> "\\n\\t"\n' >> $TMPH
>>  
>> +echo "#define EXTERN_PREFIX \"${extern_prefix}\"" >> $TMPH
>>  
>>  echo "#endif /* FFMPEG_CONFIG_H */" >> $TMPH
> 
> Let the bug reporting season begin...
> config.h on MinGW:
> 
> #define EXTERN_PREFIX ".bss b 00000000
> .data d 00000000
> .text t 00000000
> _"
> 
> Attached patch fixes it in someway, but you'll probably figure out some 
> other way.

Or this patch...

By the way, good job. It really is much simpler now. (and now there is 
only one #ifdef __MINGW32__ in all of lav*!!!).

Ramiro Polla
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ff_extern_2.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080318/9bf91632/attachment.txt>