From 52847a951802fda7a3ce36cdac77c34914b0ccca Mon Sep 17 00:00:00 2001 From: nicm Date: Sun, 15 Jan 2017 22:00:56 +0000 Subject: It is silly for cmd_list_parse to return an integer error when it could just return NULL. --- key-bindings.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'key-bindings.c') diff --git a/key-bindings.c b/key-bindings.c index 838510c6..65cc9b10 100644 --- a/key-bindings.c +++ b/key-bindings.c @@ -379,12 +379,10 @@ key_bindings_init(void) u_int i; struct cmd_list *cmdlist; char *cause; - int error; for (i = 0; i < nitems(defaults); i++) { - error = cmd_string_parse(defaults[i], &cmdlist, - "", i, &cause); - if (error != 0) + cmdlist = cmd_string_parse(defaults[i], "", i, &cause); + if (cmdlist == NULL) fatalx("bad default key"); cmdq_append(NULL, cmdq_get_command(cmdlist, NULL, NULL, 0)); cmd_list_free(cmdlist); -- cgit