# HG changeset patch
# User Bernhard Voelker <mail@bernhard-voelker.de>
# Date 1395798358 -3600
# Node ID 0a7fbe4650ff66b81faf877304041c75c01aa90e
# Parent  23530dae897fa1091cb0980ceb32af208ef8b893
strftime: wrap macros in "do {...} while(0)"

* lib/strftime.c (DO_NUMBER): Wrap multi-statement code block of
this macro in "do {...} while(0)" to prevent false use as a
single statement, e.g., in an un-braced "{}" else-block.
(DO_SIGNED_NUMBER, DO_TZ_OFFSET, DO_NUMBER_SPACEPAD): Likewise.
(strftime_case_): Remove 'else' after 'goto' - which was the
only non-fatal, un-braced use of one of the above macros.
Spotted by coverity (NESTING_INDENT_MISMATCH).

diff -r 23530dae897f -r 0a7fbe4650ff ChangeLog
--- a/ChangeLog	Wed Mar 26 01:42:11 2014 +0100
+++ b/ChangeLog	Wed Mar 26 02:45:58 2014 +0100
@@ -1,3 +1,14 @@
+2014-03-26  Bernhard Voelker  <mail@bernhard-voelker.de>
+
+	strftime: wrap macros in "do {...} while(0)"
+	* lib/strftime.c (DO_NUMBER): Wrap multi-statement code block of
+	this macro in "do {...} while(0)" to prevent false use as a
+	single statement, e.g., in an un-braced "{}" else-block.
+	(DO_SIGNED_NUMBER, DO_TZ_OFFSET, DO_NUMBER_SPACEPAD): Likewise.
+	(strftime_case_): Remove 'else' after 'goto' - which was the
+	only non-fatal, un-braced use of one of the above macros.
+	Spotted by coverity (NESTING_INDENT_MISMATCH).
+
 2014-03-26  Bernhard Voelker  <mail@bernhard-voelker.de>
 
 	modechange: avoid memory leaks for invalid octal modes
diff -r 23530dae897f -r 0a7fbe4650ff lib/strftime.c
--- a/lib/strftime.c	Wed Mar 26 01:42:11 2014 +0100
+++ b/lib/strftime.c	Wed Mar 26 02:45:58 2014 +0100
@@ -681,24 +681,44 @@
       switch (format_char)
         {
 #define DO_NUMBER(d, v) \
-          digits = d;                                                         \
-          number_value = v; goto do_number
+          do                                                                  \
+            {                                                                 \
+              digits = d;                                                     \
+              number_value = v;                                               \
+              goto do_number;                                                 \
+            }                                                                 \
+          while (0)
 #define DO_SIGNED_NUMBER(d, negative, v) \
-          digits = d;                                                         \
-          negative_number = negative;                                         \
-          u_number_value = v; goto do_signed_number
+          do                                                                  \
+            {                                                                 \
+              digits = d;                                                     \
+              negative_number = negative;                                     \
+              u_number_value = v;                                             \
+              goto do_signed_number;                                          \
+            }                                                                 \
+          while (0)
 
           /* The mask is not what you might think.
              When the ordinal i'th bit is set, insert a colon
              before the i'th digit of the time zone representation.  */
 #define DO_TZ_OFFSET(d, negative, mask, v) \
-          digits = d;                                                         \
-          negative_number = negative;                                         \
-          tz_colon_mask = mask;                                               \
-          u_number_value = v; goto do_tz_offset
+          do                                                                  \
+            {                                                                 \
+              digits = d;                                                     \
+              negative_number = negative;                                     \
+              tz_colon_mask = mask;                                           \
+              u_number_value = v;                                             \
+              goto do_tz_offset;                                              \
+            }                                                                 \
+          while (0)
 #define DO_NUMBER_SPACEPAD(d, v) \
-          digits = d;                                                         \
-          number_value = v; goto do_number_spacepad
+          do                                                                  \
+            {                                                                 \
+              digits = d;                                                     \
+              number_value = v;                                               \
+              goto do_number_spacepad;                                        \
+            }                                                                 \
+          while (0)
 
         case L_('%'):
           if (modifier != 0)
@@ -1265,9 +1285,9 @@
             }
           if (modifier == L_('O'))
             goto bad_format;
-          else
-            DO_SIGNED_NUMBER (4, tp->tm_year < -TM_YEAR_BASE,
-                              tp->tm_year + (unsigned int) TM_YEAR_BASE);
+
+          DO_SIGNED_NUMBER (4, tp->tm_year < -TM_YEAR_BASE,
+                            tp->tm_year + (unsigned int) TM_YEAR_BASE);
 
         case L_('y'):
           if (modifier == L_('E'))