[FFmpeg-cvslog] r9321 - trunk/configure

Måns Rullgård mans
Fri Jun 15 22:38:30 CEST 2007


M?ns Rullg?rd <mans at mansr.com> writes:

> Aurelien Jacobs <aurel at gnuage.org> writes:
>
>> On Fri, 15 Jun 2007 19:52:18 +0100
>> M?ns Rullg?rd <mans at mansr.com> wrote:
>>
>>> M?ns Rullg?rd <mans at mansr.com> writes:
>>> 
>>> > aurel <subversion at mplayerhq.hu> writes:
>>> >
>>> >> Author: aurel
>>> >> Date: Fri Jun 15 20:47:30 2007
>>> >> New Revision: 9321
>>> >>
>>> >> Log:
>>> >> generate ENABLE_* defines also for ARCH_*
>>> >
>>> > Who gave you permission to do this?  Stop messing around in other
>>> > people's areas without asking first.
>>> 
>>> And WTF was this change good for anyway?
>>
>> Sorry for the mess :-(
>> In fact I realized too late that my previous commit (r9320) broke
>> compilation because ENABLE_ARMV4L and other ARCH related ENABLE_*
>> wasn't defined.
>> So I commited this to fix compilation.
>> This allows to write cleaner code with if() instead of #ifdef.
>>
>> This also allows a slight simplification of configure (see attached
>> patch)
>>
>> Now do you want me to revert this commit (and the previous one),
>> or to improve it in some way ?
>
> Just leave it.  I'll tidy it up.

Done, and configure runs a bit faster too.

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




More information about the ffmpeg-cvslog mailing list