Resolve GitLab issue #3. Applies MISRA and CERT guidelines.
authorDavid James Sherman <david.sherman@inria.fr>
Wed, 21 Jun 2017 15:09:02 +0000 (17:09 +0200)
committerEric S. Raymond <esr@thyrsus.com>
Wed, 21 Jun 2017 17:14:53 +0000 (13:14 -0400)
MISRA C++:2008, 2-13-4 - Literal suffixes shall be upper case

MISRA C:2012, 7.3 - The lowercase character "l" shall not be used in a
literal suffix

CERT DCL16-C. - Use "L," not "l," to indicate a long value

CERT DCL16-CPP. - Use "L," not "l," to indicate a long value

CERT, DCL50-J. - Use visually distinct identifiers

dungeon.c

index a39e41394560b1d8f24916173a22a4e04b6a7166..e92dcc98039e55a5ecf8a3f5aadf8e9df8877670 100644 (file)
--- a/dungeon.c
+++ b/dungeon.c
@@ -54,7 +54,7 @@ long ACTSPK[VRBSIZ + 1];
 
 static bool is_set(long var, long position)
 {
 
 static bool is_set(long var, long position)
 {
-    long mask = 1l << position;
+    long mask = 1L << position;
     bool result = (var & mask) == mask;
     return (result);
 }
     bool result = (var & mask) == mask;
     return (result);
 }
@@ -308,7 +308,7 @@ static void read_conditions(FILE* database)
         while ((loc = GETNUM(NULL)) != 0) {
             if (is_set(COND[loc], K))
                 BUG(LOCATION_HAS_CONDITION_BIT_BEING_SET_TWICE);
         while ((loc = GETNUM(NULL)) != 0) {
             if (is_set(COND[loc], K))
                 BUG(LOCATION_HAS_CONDITION_BIT_BEING_SET_TWICE);
-            COND[loc] = COND[loc] + (1l << K);
+            COND[loc] = COND[loc] + (1L << K);
         }
     }
 }
         }
     }
 }