Fix bug #15565 with assertion violations in x_free_gc.

 src/xfaces.c (x_free_gc) [HAVE_X_WINDOWS, HAVE_NTGUI]: Don't pass
 expressions with side effects to eassert.

diff --git a/src/ChangeLog b/src/ChangeLog
index a205ea72b7f6e90fe71edb7f7f26080b86f67709..eee3ee8237e0299bd2b995943dbea9ba511f0631 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2013-10-09  Eli Zaretskii  <eliz@gnu.org>
+
+       * xfaces.c (x_free_gc) [HAVE_X_WINDOWS, HAVE_NTGUI]: Don't pass
+       expressions with side effects to eassert.  (Bug#15565)
+
 2013-10-09  Stefan Monnier  <monnier@iro.umontreal.ca>
 
        * fns.c (hashfn_user_defined): Allow hash functions to return any

diff --git a/src/xfaces.c b/src/xfaces.c
index 8d78a28eab10b923587519b2864c1a415f1e1b4f..7e83972955c22c5ccaa1bed203e555776b51cd2a 100644 (file)
--- a/src/xfaces.c
+++ b/src/xfaces.c
@@ -603,7 +603,7 @@ static void
 x_free_gc (struct frame *f, GC gc)
 {
   eassert (input_blocked_p ());
-  IF_DEBUG (eassert (--ngcs >= 0));
+  IF_DEBUG ((--ngcs, eassert (ngcs >= 0)));
   XFreeGC (FRAME_X_DISPLAY (f), gc);
 }
 
@@ -629,7 +629,7 @@ x_create_gc (struct frame *f, unsigned long mask, XGCValues *xgcv)
 static void
 x_free_gc (struct frame *f, GC gc)
 {
-  IF_DEBUG (eassert (--ngcs >= 0));
+  IF_DEBUG ((--ngcs, eassert (ngcs >= 0)));
   xfree (gc);
 }