[Ffmpeg-devel] [PATCH] missing parentheses in macro definition

[Guillaume Poirier]

Hi,

Marco Manfredini wrote:
> CHECK_BIDIR expands to something like --1>=0 which gcc surprisingly eats, but 
> the Intel Compiler not. 
> 
> With this change, Intel C++ 9.0 for linux/i386 compiles and passes all tests 
> with the following configuration: 
> 
> FFmpeg version SVN-r5980, Copyright (c) 2000-2004 Fabrice Bellard
>   configuration:  --disable-opts --disable-mmx --cc=icc 
> --extra-ldflags=-lguide -ldl
>   libavutil version: 49.0.0
>   libavcodec version: 51.11.0
>   libavformat version: 50.5.0
>   built on Aug 11 2006 15:16:32, gcc: Intel(R) C++ gcc 3.0 mode

That's interesting... Would you, by any chance, be working on
ultimately making FFmpeg compile and work with ICC WITH optimizations?

That would be a nice thing to do, which I have never had time to do
myself.

Other that that, your patch looks decent. Let"s see what Michael will
say about it...

Guillaume