From d4bed025ed6de39e517422b076b5af9d7b31abb7 Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Fri, 26 May 2017 13:20:20 -0400 Subject: command: Store the command parsing information directly in array Instead of defining an array of pointers, just define the array directly. Signed-off-by: Kevin O'Connor --- src/command.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'src/command.c') diff --git a/src/command.c b/src/command.c index 2a248f16..ae2b10fe 100644 --- a/src/command.c +++ b/src/command.c @@ -213,14 +213,9 @@ DECL_SHUTDOWN(sendf_shutdown); static const struct command_parser * command_get_handler(uint8_t cmdid) { - if (cmdid >= READP(command_index_size)) - goto error; - const struct command_parser *cp = READP(command_index[cmdid]); - if (!cp) - goto error; - return cp; -error: - shutdown("Invalid command"); + if (!cmdid || cmdid >= READP(command_index_size)) + shutdown("Invalid command"); + return &command_index[cmdid]; } enum { CF_NEED_SYNC=1<<0, CF_NEED_VALID=1<<1 }; -- cgit v1.2.3-70-g09d2