diff options
author | Steve Holme <steve_holme@hotmail.com> | 2014-03-22 16:31:13 +0000 |
---|---|---|
committer | Steve Holme <steve_holme@hotmail.com> | 2014-03-22 16:31:13 +0000 |
commit | 5e097583f7d5c633c939da26e7cc009d4b259b45 (patch) | |
tree | d667ee7a1d84bc0ae2733e01cf8bc97d9ad048ea /src | |
parent | 196140dcaff18675cd141487868dd61c6d0df6a7 (diff) |
tool_getparam: Removed "dead assignment" code introduced in commit 1a9b58fc
Diffstat (limited to 'src')
-rw-r--r-- | src/tool_getparam.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/tool_getparam.c b/src/tool_getparam.c index 26ec8a7a2..a0be434ca 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 */ |