Fix warnings found by clang with new warning flags to be added in later commit

Fixes consist in:
- no use of comma operator for multi statement purpose
- avoid confusing comma in for loops first and third clauses
- avoid implicit long to int casts by storing to long, or explicit bound checking before cast

1 files changed, 6 insertions, 2 deletions

diff --git a/src/proj_strtod.c b/src/proj_strtod.c
index ad197d2a..a3bc7d40 100644
--- a/src/proj_strtod.c
+++ b/src/proj_strtod.c
@@ -134,9 +134,13 @@ double proj_strtod(const char *str, char **endptr) {
     /* Then handle optional prefixed sign and skip prefix zeros */
     switch (*p) {
         case '-':
-            sign = -1,  p++;   break;
+            sign = -1;
+            p++;
+            break;
         case '+':
-            sign =  1,  p++;  break;
+            sign =  1;
+            p++;
+            break;
         default:
             if (isdigit(*p) || '_'==*p || '.'==*p)
                 break;