From: Martin Mares <mj@ucw.cz>
Date: Fri, 18 Jun 2010 20:48:14 +0000 (+0200)
Subject: UCW: Cosmetic cleanups of strtonum
X-Git-Tag: holmes-import
X-Git-Url: http://mj.ucw.cz/gitweb/?a=commitdiff_plain;h=40b53172a80fa256e45a52f6a44e55e5bc6ae94b;p=libucw.git

UCW: Cosmetic cleanups of strtonum
---

diff --git a/ucw/strtonum-gen.h b/ucw/strtonum-gen.h
index c21a9880..079548ff 100644
--- a/ucw/strtonum-gen.h
+++ b/ucw/strtonum-gen.h
@@ -7,8 +7,10 @@
  *	of the GNU Lesser General Public License.
  */
 
-/* This is not a normall header file, it is generator of a function for converting strings to integers
- * of a certain type. This file should be used only by ucw/stronum.c .
+/*
+ * This is not a normal header file, it is a generator of a function for
+ * converting strings to integers of a certain type. This file should be used
+ * by ucw/stronum.c only.
  */
 
 #define STN_DECLARE(type, suffix) STN_DECLARE_CONVERTOR(type, suffix)
@@ -27,6 +29,7 @@ STN_DECLARE(STN_TYPE, STN_SUFFIX)
 
   uns sign, base;
   err = str_to_num_init(&p, flags, &sign, &base);
+
   const char *parse_string(void)
   {
     const STN_TYPE max = STN_MAX;
@@ -98,6 +101,7 @@ STN_DECLARE(STN_TYPE, STN_SUFFIX)
 
     return NULL;
   }
+
   if (!err)
     err = parse_string();
 
diff --git a/ucw/strtonum.c b/ucw/strtonum.c
index 5a494165..43656fca 100644
--- a/ucw/strtonum.c
+++ b/ucw/strtonum.c
@@ -56,7 +56,7 @@ static const char *str_to_num_init(const char **pp, const uns flags, uns *sign,
   const char *err = NULL;
   const char *p = *pp;
 
-   // Parse sign
+  // Parse sign
   *sign = 0;
   if (flags & (STN_SIGNS))
     {
diff --git a/ucw/strtonum.h b/ucw/strtonum.h
index 28580415..597790bc 100644
--- a/ucw/strtonum.h
+++ b/ucw/strtonum.h
@@ -14,7 +14,7 @@ enum str_to_num_flags {
   STN_SIGNED = 0x20,       // The resulting range is signed
   STN_MINUS = 0x40,        // Allow optional '-' sign
   STN_PLUS = 0x80,         // Allow optional '+' sign
-  STN_TRUNC = 0x100,       // Allow range overflow -> truncate to the resulting range
+  STN_TRUNC = 0x100,       // Allow range overflow -> truncate to the allowed range
   STN_DEC = 0x200,         // Support decimal numbers
   STN_HEX = 0x400,         // Support hexadecimal numbers (0x...)
   STN_BIN = 0x800,         // Support binary numbers (0b...)
@@ -26,18 +26,18 @@ enum str_to_num_flags {
 #define STN_DBASES_MASK    0x1F
 #define STN_SIGNS          (STN_MINUS | STN_PLUS)
 #define STN_BASES          (STN_DEC | STN_HEX | STN_BIN | STN_OCT)
-#define STN_FLAGS      (STN_MINUS | STN_PLUS | STN_BASES)
-#define STN_UFLAGS     (STN_FLAGS | STN_UNDERSCORE)
-#define STN_SFLAGS     (STN_FLAGS | STN_SIGNED)
-#define STN_USFLAGS    (STN_SFLAGS | STN_UNDERSCORE)
+#define STN_FLAGS          (STN_MINUS | STN_PLUS | STN_BASES)
+#define STN_UFLAGS         (STN_FLAGS | STN_UNDERSCORE)
+#define STN_SFLAGS         (STN_FLAGS | STN_SIGNED)
+#define STN_USFLAGS        (STN_SFLAGS | STN_UNDERSCORE)
 
-#define STN_DECLARE_CONVERTOR(type, suffix)                                                         \
+#define STN_DECLARE_CONVERTOR(type, suffix)                                                               \
 const char *str_to_##suffix(type *num, const char *str, const char **next, const uns flags)
 
-#define STN_SIGNED_CONVERTOR(type, suffix, usuffix)                                                  \
+#define STN_SIGNED_CONVERTOR(type, suffix, usuffix)                                                       \
 static inline const char *str_to_##suffix(type *num, const char *str, const char **next, const uns flags) \
-{                                                                                                   \
-  return str_to_##usuffix((void *) num, str, next, flags | STN_SIGNED);                               \
+{                                                                                                         \
+  return str_to_##usuffix((void *) num, str, next, flags | STN_SIGNED);                                   \
 }
 
 STN_DECLARE_CONVERTOR(uns, uns);