Merge branch 'master' into actions-arithmetic

[open-adventure.git] / misc.c

diff --git a/misc.c b/misc.c
index 7a86b9f2a1682d104d64819d9e62dd899b51b953..243381a86330d1df2221b4f3ee6975b8b5cd6b34 100644 (file)
--- a/misc.c
+++ b/misc.c
@@ -67,7 +67,7 @@ void newspeak(char* msg)
  
   // Handle format specifiers (including the custom %C, %L, %S) by adjusting the parameter accordingly, and replacing the specifier with %s.
   int pi = 0; // parameter index
-  for (int i = 0; i < strlen(msg); ++i)
+  for (int i = 0; i < (int)strlen(msg); ++i)
     {
       if (msg[i] == '%')
        {
@@ -105,9 +105,9 @@ void newspeak(char* msg)
            {
              copy[i + 1] = 's';
              packed_to_token(PARMS[pi], parameters[pi]);
-             for (int i = 0; i < strlen(parameters[pi]); ++i)
+             for (int j = 0; j < (int)strlen(parameters[pi]); ++j)
                {
-                 parameters[pi][i] = tolower(parameters[pi][i]);
+                 parameters[pi][j] = tolower(parameters[pi][j]);
                }
            }
 
@@ -116,9 +116,9 @@ void newspeak(char* msg)
            {
              copy[i + 1] = 's';
              packed_to_token(PARMS[pi], parameters[pi]);
-             for (int i = 0; i < strlen(parameters[pi]); ++i)
+             for (int j = 0; j < (int)strlen(parameters[pi]); ++j)
                {
-                 parameters[pi][i] = tolower(parameters[pi][i]);
+                 parameters[pi][j] = tolower(parameters[pi][j]);
                }
              parameters[pi][0] = toupper(parameters[pi][0]);
            }
@@ -196,7 +196,7 @@ bool GETIN(FILE *input,
            (junk > 0);
        if (GETTXT(true,true,true) <= 0)
            return true;
-       RSPEAK(ARB_53);
+       RSPEAK(TWO_WORDS);
     }
 }
 
@@ -217,7 +217,7 @@ long YES(FILE *input, vocab_t x, vocab_t y, vocab_t z)
            RSPEAK(z);
            return false;
        }
-       RSPEAK(ARB_185);
+       RSPEAK(PLEASE_ANSWER);
     }
 }
 
@@ -316,16 +316,16 @@ long VOCAB(long id, long init)
  *  (Thus "STEPS", which is a motion verb as well as an object, may be located
  *  as an object.)  And it also means the KTAB value is taken modulo 1000. */
 {
-    long i, lexeme;
+    long lexeme;
 
-    for (i=1; i<=TABSIZ; i++) {
+    for (long i=1; i<=TABSIZ; i++) {
        if (KTAB[i] == -1) {
            lexeme= -1;
            if (init < 0)
                return(lexeme);
            BUG(5);
        }
-       if (init >= 0 && KTAB[i]/1000 != init) 
+       if (init >= 0 && KTAB[i]/1000 != init)
            continue;
        if (ATAB[i] == id) {
            lexeme=KTAB[i];
@@ -427,13 +427,13 @@ long ATDWRF(long where)
  *  there (or if dwarves not active yet), -1 if all dwarves are dead.  Ignore
  *  the pirate (6th dwarf). */
 {
-    long at, i;
+    long at;
 
     at =0;
     if (game.dflag < 2)
        return(at);
     at = -1;
-    for (i=1; i<=NDWARVES-1; i++) {
+    for (long i=1; i<=NDWARVES-1; i++) {
        if (game.dloc[i] == where)
            return i;
        if (game.dloc[i] != 0)