[PATCH] Fix make closures.c use dlmmap,dlmunmap on cygwin.

Dave Korn dave.korn.cygwin@googlemail.com
Tue Jul 7 13:52:00 GMT 2009


[ Re: http://gcc.gnu.org/ml/gcc-patches/2009-06/threads.html#02228 ]

    Hey all,

  Take 2 of this patch.  Works in the simplest way possible, by saying that
cygwin is an exception to the rule about win32/64 platforms not using the
local dlmmap/dlmunmap implementations.

  Sticklers for formatting will note that two of the #endif lines reach 81
characters, because of the comment repeating the #if condition.  I didn't
think it was worth wrapping the lines for such a tiny excess length in
something this trivial, but I'll do so before committing if that's what the
maintainers decide.

libffi/ChangeLog:

	* closures.c (mmap, munmap):  Don't define replacement macros pointing
	to dl* versions on windows platforms.

  Bootstrapping now.  Assuming it completes and libffi tests run alright, OK?

    cheers,
      DaveK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libffi-cygwin-mmap-fix.diff
Type: text/x-c
Size: 1410 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20090707/2590be29/attachment.bin>


More information about the Java-patches mailing list