[FFmpeg-devel] Suggestion: revert r12154
Mon Feb 25 14:53:44 CET 2008
On Mon, Feb 25, 2008 at 7:35 PM, M?ns Rullg?rd <mans at mansr.com> wrote:
> Reimar D?ffinger wrote:
> > On Mon, Feb 25, 2008 at 06:26:51PM +0800, mvplayer wrote:
> >> avutil.h is a public file which should not contain any gcc
> >> definition!
> > I assume you mean gcc-specific? If there is something gcc-specific that
> > definitely is wrong, on the other hand if this is only Microsoft's
> > compiler not even getting the basics of the C standard right you would
> > have to find and suggest a clean solution yourself...
> There is nothing gcc-specific in avutil.h, or any other installed header.
> The macros under discussion are perfectly valid C. The standard says
> the expansion of a macro should be reparsed as source. Apparently,
> msvc does not do this correctly.
> I have verified these macros to work with a variety of non-gcc compilers.
> None of them had any problem at all expanding the macros.
> I'm sorry, but you need a C compiler to use the FFmpeg headers. They
> won't work with Fortran, Pascal, Ada, or Java compilers, so why should
> they be expected to work with msvc?
Do you mean msvs is not a C compiler?
It is not same as java, but it is a C compiler
r12154 introduce more complicated code, why do you submit the dirty so
called "clean up " modifications
> Inspired by http://www.mvplayer.net
More information about the ffmpeg-devel