[Ffmpeg-devel] [PATCH] Add allformats.h include file to move *_init() functions and other

Steve Lhomme slhomme
Thu Jun 29 19:35:06 CEST 2006


Rich Felker wrote:
> On Thu, Jun 29, 2006 at 07:19:30PM +0200, Diego 'Flameeyes' Petten? wrote:
>> On Thursday 29 June 2006 19:06, Rich Felker wrote:
>>> because it's part of the c++monkey mindset to add it to every header
>>> file...
>> Most likely because I simply put it when I'm not sure if the code ends up 
>> being standalone or what. FFmpeg is currently being used/imported by other 
>> projects, and "better safe than sorry" is my usual line here (especially 
>> considering it's being used by wengophone that uses scons, which I don't 
>> trust for correctly separate C and C++ code for instance).
> 
> extern "C" { } does not even belong in headers to be used by the
> caller. instead, c++ apps should put their own extern "C" {} around
> includes for C-language headers. libavcodec is written in C, not C++,
> and extern "C" {} is not part of C.

Is there a C compiler that fails on extern "C" ? At least among the 
one(s) supported by FFMPEG.

Steve





More information about the ffmpeg-devel mailing list