From 5e097583f7d5c633c939da26e7cc009d4b259b45 Mon Sep 17 00:00:00 2001 From: Steve Holme Date: Sat, 22 Mar 2014 16:31:13 +0000 Subject: [PATCH] tool_getparam: Removed "dead assignment" code introduced in commit 1a9b58fc --- src/tool_getparam.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/tool_getparam.c b/src/tool_getparam.c index 26ec8a7..a0be434 100644 --- a/src/tool_getparam.c +++ b/src/tool_getparam.c @@ -1828,6 +1828,10 @@ ParameterError parse_args(struct GlobalConfig *config, int argc, result = getparameter(flag, nextarg, &passarg, config, operation); if(result == PARAM_NEXT_OPERATION) { + /* Reset result as PARAM_NEXT_OPERATION is only used here and not + returned from this function */ + result = PARAM_OK; + if(operation->url_list && operation->url_list->url) { /* Allocate the next config */ operation->next = malloc(sizeof(struct OperationConfig)); @@ -1851,9 +1855,6 @@ ParameterError parse_args(struct GlobalConfig *config, int argc, else result = PARAM_NO_MEM; } - - /* Reset result to continue */ - result = PARAM_OK; } else if(!result && passarg) i++; /* we're supposed to skip this */ -- 2.7.4