tree 32f87dd85aa3b0d58fb45bbef60591ee7c43dce8
parent c9c06d0d8cbc766374c43328e91b7a87f93e9c62
author David Lamparter <equinox@opensourcerouting.org> 1431443884 +0200
committer David Lamparter <equinox@opensourcerouting.org> 1431606390 +0200

lib: fix "reduce strcmp in CLI" fallout (10bac801)

In "lib/cli: reduce strcmp in CLI hot paths", I failed to notice that
CMD_VARIABLE as a boolean test covers a superset of the other types of
variables.  Thus, the patch broke processing of IP/IPv6/Integer range
parameters in the CLI.

Fix by some reordering and introducing TERMINAL_RECORD macro (which
marks whether a given terminal type is a parameter) to be used in places
where the check is really for all kinds of variables.

Reported-by: Timo Teräs <timo.teras@iki.fi>
Tested-by: Martin Winter <mwinter@netdef.org>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>