Unified terminology: arguments, not parameters

diff --git a/TODO b/TODO
index e0f9465351164a77d0f46dd577d8fd1b9fa08cf4..48bcd544daa389e504c74ac42248086ff690d766 100644 (file)
--- a/TODO
+++ b/TODO
@@ -2,7 +2,6 @@
 - What if an input page specifies /Rotate?
 - "-f" switch
 - Help
-- Parameters vs. arguments
 - More precise parsing errors
 
 | # Position bbox on a new paper

diff --git a/cmds.cc b/cmds.cc
index ece509d8cfb9efa6c5a16c1175011cb196b0f461..e446c8392b889481b7d6c53f39d8d2d79c20f44e 100644 (file)
--- a/cmds.cc
+++ b/cmds.cc
@@ -1079,7 +1079,7 @@ public:
       else if (by == "tile" || by == "t")
        fill_by = BY_TILE;
       else
-       err("Parameter \"by\" must be rows/cols/tile");
+       err("Argument \"by\" must be rows/cols/tile");
 
       crop = c->arg("crop")->as_int(0);
       mixed = c->arg("mixed")->as_int(0);

diff --git a/parse.cc b/parse.cc
index 5f1b1d035c119ee1a729e5f1900f1232b288e80a..37ee377eaae4fae0d2dfdfefc20231a625c67897 100644 (file)
--- a/parse.cc
+++ b/parse.cc
@@ -214,7 +214,7 @@ static double parse_dimen(const arg_def *adef)
 {
   token_type t = next_token();
   if (t != TOK_NUMBER)
-    parse_error("Parameter %s must be a dimension", adef->name);
+    parse_error("Argument %s must be a dimension", adef->name);
   double tmp = token_num;
 
   t = next_token();
@@ -225,7 +225,7 @@ static double parse_dimen(const arg_def *adef)
          return_token();
          return 0;
        }
-      parse_error("Parameter %s must have a unit", adef->name);
+      parse_error("Argument %s must have a unit", adef->name);
     }
   for (uint i=0; units[i].name; i++)
     if (token == units[i].name)
@@ -316,9 +316,9 @@ static void parse_args(cmd *c)
          while (adefs[argi].name && token != adefs[argi].name)
            argi++;
          if (!adefs[argi].name)
-           parse_error("Command %s has no parameter %s", cdef->name, token.c_str());
+           parse_error("Command %s has no argument %s", cdef->name, token.c_str());
          if (c->args.count(token))
-           parse_error("Parameter %s given multiple times", token.c_str());
+           parse_error("Argument %s given multiple times", token.c_str());
          t = next_token();
          if (t == TOK_EQUAL)
            has_value = true;
@@ -327,7 +327,7 @@ static void parse_args(cmd *c)
          saw_named = true;
        }
       else if (saw_named)
-       parse_error("Positional parameters must precede named ones");
+       parse_error("Positional arguments must precede named ones");
       else
        {
          return_token();
@@ -349,19 +349,19 @@ static void parse_args(cmd *c)
            case AT_STRING:
              t = next_token();
              if (t != TOK_STRING)
-               parse_error("Parameter %s must be a string", adef->name);
+               parse_error("Argument %s must be a string", adef->name);
              val = new arg_string(token);
              break;
            case AT_INT:
              t = next_token();
              if (t != TOK_NUMBER || !token_is_int())
-               parse_error("Parameter %s must be an integer", adef->name);
+               parse_error("Argument %s must be an integer", adef->name);
              val = new arg_int((int) token_num);
              break;
            case AT_DOUBLE:
              t = next_token();
              if (t != TOK_NUMBER)
-               parse_error("Parameter %s must be a number", adef->name);
+               parse_error("Argument %s must be a number", adef->name);
              val = new arg_double(token_num);
              break;
            case AT_DIMEN:
@@ -370,7 +370,7 @@ static void parse_args(cmd *c)
            case AT_SWITCH:
              t = next_token();
              if (t != TOK_NUMBER || !token_is_int() || ((int) token_num != 0 && (int) token_num != 1))
-               parse_error("Parameter %s must be a switch", adef->name);
+               parse_error("Argument %s must be a switch", adef->name);
              val = new arg_int((int) token_num);
              break;
            default:
@@ -382,7 +382,7 @@ static void parse_args(cmd *c)
          if (type == AT_SWITCH)
            val = new arg_int(1);
          else
-           parse_error("Parameter %s must have a value", adef->name);
+           parse_error("Argument %s must have a value", adef->name);
        }
 
       c->args[adef->name] = val;
@@ -391,12 +391,12 @@ static void parse_args(cmd *c)
       if (t == TOK_CLOSE_PAREN)
        break;
       if (t != TOK_COMMA)
-       parse_error("Comma expected after parameter %s", adef->name);
+       parse_error("Comma expected after argument %s", adef->name);
     }
 
   for (uint i=0; i<num_args; i++)
     if ((adefs[i].type & AT_MANDATORY) && !c->args.count(adefs[i].name))
-      parse_error("Command %s is missing a parameter %s", cdef->name, adefs[i].name);
+      parse_error("Command %s is missing an argument %s", cdef->name, adefs[i].name);
 }
 
 static void debug_cmd(cmd *c, uint indent=0)