[FFmpeg-cvslog] r9321 - trunk/configure

Måns Rullgård mans
Fri Jun 15 21:13:44 CEST 2007


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.

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




More information about the ffmpeg-cvslog mailing list