"Ronald S. Bultje" <rsbul...@gmail.com> writes:

> Hi,
>
> 2012/1/23 Måns Rullgård <m...@mansr.com>:
>> "Ronald S. Bultje" <rsbul...@gmail.com> writes:
>>
>>> This allows combining multiple conditionals in a single statement.
>>> ---
>>>
>>> diff --git a/configure b/configure
>>> index 3146641..bd3e4f1 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -542,7 +542,8 @@ print_config_mak(){
>>>  }
>>>
>>>  print_config_asm(){
>>> -    enabled $1 && echo "%define $2"
>>> +    enabled $1 && v=1 || v=0
>>> +    echo "%define $2 $v"
>>>  }
>>
>> Why did we use %ifdef originally?  Are there compatibility problems with
>> nasm or some widely installed yasm version?  I vaguely recall some
>> discussion about this back when config.asm was added.
>
> Likely because that's what we used before config.asm existed,

config.asm was added after we changed config.h to #define FOO 0/1.

> and that was because that's what x264 used.

I can't find any traces of such a discussion in the list archives, so I
must have imagined it.  The change should be OK then.

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to