aboutsummaryrefslogtreecommitdiff
path: root/common/cmd_itest.c
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2010-10-20 07:17:33 -0400
committerWolfgang Denk <wd@denx.de>2010-11-28 21:58:26 +0100
commitfc9903f38d53024e28261eb14230bf6fc02d879e (patch)
tree40f4bfb52e8ffc31251e6504be09f3d94cc93de0 /common/cmd_itest.c
parentbdbc1303cb6a3ddbbf7a4f6a546f8aa85775a13e (diff)
cmd_itest: constify & localize op table
No one else needs this table. While we're here, use the standard ARRAY_SIZE helper macro. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Diffstat (limited to 'common/cmd_itest.c')
-rw-r--r--common/cmd_itest.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/common/cmd_itest.c b/common/cmd_itest.c
index 8dd8927b5..fa6a0c301 100644
--- a/common/cmd_itest.c
+++ b/common/cmd_itest.c
@@ -46,7 +46,7 @@ struct op_tbl_s {
typedef struct op_tbl_s op_tbl_t;
-op_tbl_t op_table [] = {
+static const op_tbl_t op_table [] = {
{ "-lt", LT },
{ "<" , LT },
{ "-gt", GT },
@@ -62,8 +62,6 @@ op_tbl_t op_table [] = {
{ "<=" , LE },
};
-#define op_tbl_size (sizeof(op_table)/sizeof(op_table[0]))
-
static long evalexp(char *s, int w)
{
long l = 0;
@@ -138,12 +136,12 @@ static int arithcomp (char *s, char *t, int op, int w)
int binary_test (char *op, char *arg1, char *arg2, int w)
{
int len, i;
- op_tbl_t *optp;
+ const op_tbl_t *optp;
len = strlen(op);
for (optp = (op_tbl_t *)&op_table, i = 0;
- i < op_tbl_size;
+ i < ARRAY_SIZE(op_table);
optp++, i++) {
if ((strncmp (op, optp->op, len) == 0) && (len == strlen (optp->op))) {