[Ffmpeg-devel] [PATCH] rename CONFIG_WIN32 to CONFIG_MINGW/__MINGW32__

Diego Biurrun diego
Wed Jul 12 13:09:42 CEST 2006


On Tue, Jul 11, 2006 at 03:12:51PM +0200, Diego Biurrun wrote:
> On Tue, Jul 11, 2006 at 11:30:52AM +0100, M?ns Rullg?rd wrote:
> > 
> > Diego Biurrun said:
> > > On Mon, Jul 10, 2006 at 07:39:43PM +0100, M?ns Rullg?rd wrote:
> > >>
> > >> Fine by me in general, assuming the compiler defines __MINGW32__.
> > >
> > > We define it in configure.
> > 
> > No, we don't, and we must not define anything starting with and underscore.
> > Such names are reserved for the system.  GCC seems to define __MING32__
> > automatically though.
> 
> Maybe we must not, but we sure do, look at configure line 1924:
> 
> if test "$mingw32" = "yes" ; then
> [...]
> echo "#ifndef __MINGW32__" >> $TMPH
> echo "#define __MINGW32__ 1" >> $TMPH
> 
> OK, this may be another bug then ...

Is this possibly needed for crosscompilation?  Since it's protected by
#ifndef it's probably not a big deal.

> Dunno if this simplifies much, but I'll commit both patches when I come
> home this evening.

Patch committed.

Diego




More information about the ffmpeg-devel mailing list