diff options
| author | Jan Djärv | 2008-02-07 12:47:00 +0000 |
|---|---|---|
| committer | Jan Djärv | 2008-02-07 12:47:00 +0000 |
| commit | 92413ef3a119c29af24901e66b393d3c69cc83b5 (patch) | |
| tree | 597e3de17cbc3588975e040fdc239331b439d066 /src/ChangeLog | |
| parent | 03fa5713cc5eb2226a5ca3fb1ce59a9e9f46ffae (diff) | |
| download | emacs-92413ef3a119c29af24901e66b393d3c69cc83b5.tar.gz emacs-92413ef3a119c29af24901e66b393d3c69cc83b5.zip | |
Use "do...while (0)", not "if (1)..else" in macro definitions.
Diffstat (limited to 'src/ChangeLog')
| -rw-r--r-- | src/ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 9fcc90c2391..396ce321f72 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,5 +1,15 @@ | |||
| 1 | 2008-02-07 Jim Meyering <meyering@redhat.com> | 1 | 2008-02-07 Jim Meyering <meyering@redhat.com> |
| 2 | 2 | ||
| 3 | Use "do...while (0)", not "if (1)..else" in macro definitions. | ||
| 4 | The latter provokes a warning from gcc about the empty else, when | ||
| 5 | followed by ";". Also, without that trailing semicolon, it would | ||
| 6 | silently swallow up any following statement. | ||
| 7 | * syntax.h (SETUP_SYNTAX_TABLE): | ||
| 8 | (SETUP_SYNTAX_TABLE_FOR_OBJECT): Likewise. | ||
| 9 | * buffer.h (DECODE_POSITION): Likewise. | ||
| 10 | |||
| 11 | 2008-02-07 Jim Meyering <meyering@redhat.com> | ||
| 12 | |||
| 3 | * lread.c [lint]: Don't include <sys/inode.h>. | 13 | * lread.c [lint]: Don't include <sys/inode.h>. |
| 4 | 14 | ||
| 5 | 2008-02-07 Stefan Monnier <monnier@iro.umontreal.ca> | 15 | 2008-02-07 Stefan Monnier <monnier@iro.umontreal.ca> |