tree 2cb55fef6173afd9543e1af71da4dd4e7bca6c02
parent 1ab859ddee6b0017b17d9e89e195a48ddb4c4fdc
author Phillip Lougher <phillip@squashfs.org.uk> 1409882457 +0100
committer Mohamad Ayyash <mkayyash@google.com> 1424723805 -0800

actions: update test function parser to handle variable amount of arguments

Update the test function parser to handle variable amount of arguments,
as that's now needed for the perm() test function.  Up till now, though
updating the test parser has been a TODO, it been a low priority because
it's done what's needed.

Updating the parser to speculatively read the number of arguments
present (and hence handle variable number of arguments) has been a
TODO even without the need to handle a variable number of arguments
because experience gained updating the action parser to do this showed
it generates better syntax error messages.

Previously the parser just parsed the number of arguments expected, and
then quit.  The parser now parses all the arguments present, and then
compares it against what was expected.

No big deal?  Well guess what gives the better syntax error if you
enter too many arguments.  Reading just the number of arguments
expected, and then quitting, means the extra arguments cause to
paraphrase a "I expected a ) here, and I got a comma or string".
Now, you get an error message like "I got 5 arguments, but only
expected 2".  Which IMHO is a better syntax error message.

Signed-off-by: Phillip Lougher <phillip@squashfs.org.uk>