[MPlayer-dev-eng] man_anonymous

Trent Piepho xyzzy at speakeasy.org
Sun Mar 6 20:43:38 CET 2005


On Sun, 6 Mar 2005, Reimar Döffinger wrote:
> > MAP_ANONYMOUS is defined in sys/mman.h, which is only included if config.h
> > defines HAVE_SYS_MMAN_H.  Since config.h must be before sys/mman.h,
> > MAP_ANONYMOUS will never be defined and the ifndef check is pointless. 
> > config.h will always define MAP_ANONYMOUS, which will be re-defined when
> > sys/mman.h is included on systems that have MAP_ANONYMOUS (like Linux).
> 
> Fixed in a different way some time ago...

Two weeks after I sent that patch actually.

Isn't adding a bunch of ifdefs into all code that uses mmap a lot more
intrusive than a configure test that just puts the code, when necessary, into
config.h?  Isn't that the point of config.h?





More information about the MPlayer-dev-eng mailing list