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

Ismail Dönmez ismail
Fri Jan 26 12:29:20 CET 2007


On Friday 26 January 2007 11:23:34 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

I would be against this, its just breaking working functionality.

Regards,
ismail




More information about the ffmpeg-devel mailing list