[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