From: NHOrus <jy6x2b32pie9@yahoo.com>
Date: Mon, 17 Jul 2017 16:59:55 +0000 (+0300)
Subject: No magic, revert some experimental changes in pipeline
X-Git-Tag: 1.3~55
X-Git-Url: https://jxself.org/git/?a=commitdiff_plain;h=30822d0ba4af115fba437b348eac87ffb2e0241d;p=open-adventure.git

No magic, revert some experimental changes in pipeline
---

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 984111d..d337d0e 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -19,7 +19,6 @@ binary:debug:
       - "*.o"
       - dungeon.c
       - dungeon.h
-      - "*.gcno"
 
 binary:release:
   stage: build
@@ -72,16 +71,12 @@ test:debug:
   before_script:
     - zypper install -y make gcc libedit-devel libasan3 libubsan0 python python2-PyYAML lcov
   script:
-    - make debug
-    - make check
-    - lcov -t "advent" -o advent.info -c -d .
-    - genhtml -o coverage advent.info
-    - cd tests
-    - ./coverage_dungeon.py
-    - cd ..
+    - make coverage
   artifacts:
     paths:
       - coverage
+  dependencies:
+    - binary:debug
 
 test:release:
   stage: test
diff --git a/main.c b/main.c
index 350c4ae..192d129 100644
--- a/main.c
+++ b/main.c
@@ -137,7 +137,7 @@ static bool fallback_handler(struct command_t command)
 /* fallback handler for commands not handled by FORTRANish parser */
 {
     long sv;
-    char buf[2 * LINESIZE + 1];
+    char buf[DIM(command.raw1) + DIM(command.raw2) + 1];
     sprintf(buf, "%s %s", command.raw1, command.raw2);
 
     if (sscanf(buf, "seed %ld", &sv) == 1) {
@@ -146,8 +146,7 @@ static bool fallback_handler(struct command_t command)
         // autogenerated, so don't charge user time for it.
         --game.turns;
         return true;
-    }
-    else if (sscanf(buf, "waste %ld", &sv) == 1) {
+    } else if (sscanf(buf, "waste %ld", &sv) == 1) {
         game.limit -= sv;
         printf("Game limit is now %ld\n", game.limit);
         return true;
@@ -909,10 +908,10 @@ static void lampcheck(void)
             rspeak(REPLACE_BATTERIES);
             game.prop[BATTERY] = DEAD_BATTERIES;
 #ifdef __unused__
-	    /* This code from the original game seems to have been faulty.
-	     * No tests ever passed the guard, and with the guard removed
-	     * the game hangs when the lamp limit is reached.  
-	     */
+            /* This code from the original game seems to have been faulty.
+             * No tests ever passed the guard, and with the guard removed
+             * the game hangs when the lamp limit is reached.
+             */
             if (TOTING(BATTERY))
                 drop(BATTERY, game.loc);
 #endif