[Ffmpeg-devel] Re: [PATCH] Dead code cleanup - C++

Stefan Heinzmann stefan_heinzmann
Fri Jan 26 14:11:22 CET 2007


Michael Niedermayer wrote:
> Hi
> 
> On Fri, Jan 26, 2007 at 05:15:07AM +0100, Diego Biurrun wrote:
>> On Fri, Jan 26, 2007 at 01:24:39PM +1030, Yuri Vilmanis wrote:
>>> Removed some dead code relating to C++. As no part of the codebase
>>> will ever be seen by a C++ compiler, any code protected by
>>> #ifdef__cplusplus will *never* get past the preprocessor, and so can
>>> be safely removed. The files in which these guards appear are not
>>> valid C++ anyway, so removing these references to C++ should reduce
>>> future confusion on this issue. The C++ wrapper "fobs" (or other
>>> C++ wrappers I'm not aware of) can be used by anyone requiring C++
>>> support.
>> OK to apply this patch?
> 
> all the versions should be bumped a little at least so user apps
> could detect if these macros are there or not
> 
> now about the patch itself, i have no real oppinion on this, it where
> c++ people who wanted it, and now its one c++ developer who wants it
> removed, i think that the people who wanted this (see svnlog i dont
> remember at all) should at least get a chance to comment first

There is no real need for C++ users to have those lines in the header
files, as they can be provided externally (surrounding the #include
line). People who are currently using ffmpeg together with C++ will,
however, rely on this being there, so the patch will break their code.
As with all header changes that have a chance to break things, I would
propose to delay application of the patch until the version numbers get
incremented.

Cheers
Stefan





More information about the ffmpeg-devel mailing list