Revert last change to gmalloc.c; update some comments.

diff --git a/ChangeLog b/ChangeLog
index 8e8349aa487bc8dfcb6e22953f88295773b665e2..ac96bed298378ce787c9f0657c3026ad62580df4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2013-08-15  Ken Brown  <kbrown@cornell.edu>
+
+       * configure.ac (G_SLICE_ALWAYS_MALLOC): Update comment.
+
 2013-08-15  Glenn Morris  <rgm@gnu.org>
 
        * make-dist: Do not distribute etc/refcards TeX intermediate files.

diff --git a/configure.ac b/configure.ac
index 61ea613e2878aefe45c5e8f98516d4f0a3046f92..980c36f75c74df02e8feec49f5dc7ce5a02355b0 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -4340,14 +4340,16 @@ fi
 
 
 case $opsys in
-  dnl Emacs supplies its own malloc, but glib (part of Gtk+) calls
-  dnl memalign and on Cygwin, that becomes the Cygwin-supplied memalign.
-  dnl As malloc is not the Cygwin malloc, the Cygwin memalign always
-  dnl returns ENOSYS.  A workaround is to set G_SLICE=always-malloc. */
+  dnl Emacs supplies its own malloc, but glib calls posix_memalign,
+  dnl and on Cygwin prior to version 1.7.24 that becomes the
+  dnl Cygwin-supplied posix_memalign.  As malloc is not the Cygwin
+  dnl malloc, the Cygwin posix_memalign always returns ENOSYS.  A
+  dnl workaround is to set G_SLICE=always-malloc.  This is no longer
+  dnl needed starting with cygwin-1.7.24, and it is no longer
+  dnl effective starting with glib-2.36. */
   cygwin)
     AC_DEFINE(G_SLICE_ALWAYS_MALLOC, 1, [Define to set the
-      G_SLICE environment variable to "always-malloc" at startup, if
-      using GTK.])
+      G_SLICE environment variable to "always-malloc" at startup.])
     ;;
 
   hpux11)

diff --git a/src/ChangeLog b/src/ChangeLog
index 46591eb1952c6af18c77b3423ac65f7b4c5a67dd..c12b32ebc714b89af5630ed2f8ed8df4b075d5b9 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,9 @@
+2013-08-15  Ken Brown  <kbrown@cornell.edu>
+
+       * emacs.c (main): Update comment about G_SLICE_ALWAYS_MALLOC.
+       * gmalloc.c (memalign) [CYGWIN]: Revert last change; it's not
+       needed.
+
 2013-08-15  Paul Eggert  <eggert@cs.ucla.edu>
 
        Fix minor problems found by static checking.

diff --git a/src/emacs.c b/src/emacs.c
index 3c80d3ed753d70b7f45dcd2e862278d5b8d8b0c7..0538414533085ccc51bb7db6b4ec85d021ac61b5 100644 (file)
--- a/src/emacs.c
+++ b/src/emacs.c
@@ -696,7 +696,8 @@ main (int argc, char **argv)
 #endif
 
 #ifdef G_SLICE_ALWAYS_MALLOC
-  /* This is used by the Cygwin build.  */
+  /* This is used by the Cygwin build.  It's not needed starting with
+     cygwin-1.7.24, but it doesn't do any harm.  */
   xputenv ("G_SLICE=always-malloc");
 #endif
 

diff --git a/src/gmalloc.c b/src/gmalloc.c
index 42ac0b039854abae1d4abc1efb6835e7671b532c..bc1d85ac5fbb0ac1ae34f1b3f606d394f9d81388 100644 (file)
--- a/src/gmalloc.c
+++ b/src/gmalloc.c
@@ -1558,15 +1558,6 @@ License along with this library.  If not, see <http://www.gnu.org/licenses/>.  *
 
 void *(*__memalign_hook) (size_t size, size_t alignment);
 
-/* As of version 1.7.24, Cygwin allows applications to provide their
-   own posix_memalign (but not memalign).  But posix_memalign as
-   defined in this file calls memalign, so we have to rename the
-   latter in order to make sure that posix_memalign calls Emacs's
-   memalign.  */
-#ifdef CYGWIN
-#define memalign emacs_memalign
-#endif
-
 void *
 memalign (size_t alignment, size_t size)
 {