[MPlayer-dev-eng] swab is in string.h on win32 and declaration conflicts

[Diego Biurrun]

On Sun, May 22, 2011 at 08:18:53PM +0200, Reimar Döffinger wrote:
> On Sun, May 22, 2011 at 03:16:08PM +0200, Diego Biurrun wrote:
> > On Sat, May 21, 2011 at 12:36:37AM +0300, Ivan Kalvachev wrote:
> > > On 5/18/11, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> > > > Gianluigi Tiesi <mplayer <at> netfarm.it> writes:
> > > >
> > > >> on mingw32 swab is defined is string.h thus not detected by configure
> > > >> that looks in unistd.h
> > > >
> > > > http://thread.gmane.org/gmane.comp.video.mplayer.cvs/17298
> > > 
> > > Will you revert this revert or do you prefer I to be the one to do it?
> > 
> > Neither of you will.
> 
> That's not a helpful tone either. I think it is a good idea to get
> feedback from the people actually compiling against MinGW before
> deciding anything.
> If it's simple to update MinGW64 to a version that works I agree it's
> better not to keep hacks around, but I don't know how easily that is
> possible (MinGW as in non-64 is a incredibly huge pain to update and
> I wouldn't want to force that on anyone just for saving one line in
> configure and "in principle correct".

AFAIU MinGW64 is still beta-quality software and subject to changes
on a frequent basis as this example demonstrates.  I already went out
of my way to get this problem fixed at the source instead of just
blindly adding a local workaround.

Short-term pain, long-term gain.

Diego