[MPlayer-cvslog] r25592 - trunk/configure

[Diego Biurrun]

On Wed, Jan 02, 2008 at 10:56:24PM -0500, Rich Felker wrote:
> On Thu, Jan 03, 2008 at 02:44:58AM +0100, uau wrote:
> > 
> > Log:
> > Add HAVE_SOCKLEN_T to config.h for FFmpeg
> > 
> > Needed to fix compilation after recent FFmpeg changes. It's now always
> > set to true without any tests. I don't expect this to cause problems
> > as common systems will have the type and the FFmpeg demuxers which
> > would use it are not compiled under MPlayer (compilation was broken
> > because the type was redefined in a header).
> 
> I have a proposal for ugly things like this. Instead of HAVE_FOO for
> _standard_ features which should always be expected to be present by
> default, the macro should instead be MISSING_FOO. Then the code will
> compile correctly even with an empty config.h on sane systems, and
> only broken systems may encounter problems.

Funny, I was thinking that even before you wrote it.  I'll talk it over
with Mans.

Diego