[FFmpeg-cvslog] r9321 - trunk/configure

Aurelien Jacobs aurel
Fri Jun 15 21:00:30 CEST 2007


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 ?

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: simplify_target_arch.diff
Type: text/x-diff
Size: 3975 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070615/041dc87d/attachment.diff>



More information about the ffmpeg-cvslog mailing list