[FFmpeg-devel] [PATCH] avoid using cpp directives within printf macro arguments

leon l.vanstuivenberg at chello.nl
Mon Oct 29 09:27:15 CET 2012


On 29/10/2012 01:09, Paul B Mahol wrote:
>> and the inability of some older gcc versions to compile cmdutils.c
>
> And with this patch they can compile whole ffmpeg?
> Otherwise patch as is, is bloat.

This patch is one of a few changes that are needed to get ffmpeg 
compiling with gcc 3.0.3 on PS2Linux ; probably not all changes outside 
the ps2 specific code will be acceptable for the main tree, but i 
figured this one would be a reasonable compromise between maintaining it 
in a separate repository vs adding a few redundant lines compared to the 
construct which uses the gcc specific feature (which btw is not listed in
http://ffmpeg.org/developer.html#C-language-features
). IMHO, it doesn't hurt readability, and avoids the use of a 
nonstandard construct.

<leon/>



More information about the ffmpeg-devel mailing list