changeset 91619:eeab8708c9c3

*** empty log message ***
author Jan Djärv <jan.h.d@swipnet.se>
date Thu, 07 Feb 2008 12:47:00 +0000
parents 400a54e167c9
children acb32817f0e8
files src/ChangeLog
diffstat 1 files changed, 10 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/ChangeLog	Thu Feb 07 12:46:39 2008 +0000
+++ b/src/ChangeLog	Thu Feb 07 12:47:00 2008 +0000
@@ -1,3 +1,13 @@
+2008-02-07  Jim Meyering  <meyering@redhat.com>
+
+        Use "do...while (0)", not "if (1)..else" in macro definitions.
+        The latter provokes a warning from gcc about the empty else, when
+        followed by ";".  Also, without that trailing semicolon, it would
+        silently swallow up any following statement.
+        * syntax.h (SETUP_SYNTAX_TABLE):
+        (SETUP_SYNTAX_TABLE_FOR_OBJECT): Likewise.
+        * buffer.h (DECODE_POSITION): Likewise.
+
 2008-02-07  Jim Meyering  <meyering@redhat.com>
 
 	* lread.c [lint]: Don't include <sys/inode.h>.