[FFmpeg-cvslog] r9321 - trunk/configure
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
> 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.
mans at mansr.com
More information about the ffmpeg-cvslog