diff options
author | Daniel Stenberg <daniel@haxx.se> | 2016-12-14 01:29:44 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2016-12-14 01:29:44 +0100 |
commit | 1c3e8bbfedcd3822aeb1bab22fb56c5ecff4295b (patch) | |
tree | c1606588aeae4535f0faa7942fcbe50e6e340f8b /src/tool_getparam.c | |
parent | b228d2952b6762b5c9b851fba0cf391e80c6761a (diff) |
checksrc: warn for assignments within if() expressions
... they're already frowned upon in our source code style guide, this
now enforces the rule harder.
Diffstat (limited to 'src/tool_getparam.c')
-rw-r--r-- | src/tool_getparam.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/tool_getparam.c b/src/tool_getparam.c index 247f8d38c..9b5c9a54c 100644 --- a/src/tool_getparam.c +++ b/src/tool_getparam.c @@ -757,7 +757,11 @@ ParameterError getparameter(char *flag, /* f or -long-flag */ case '@': /* the URL! */ { struct getout *url; - if(config->url_get || ((config->url_get = config->url_list) != NULL)) { + + if(!config->url_get) + config->url_get = config->url_list; + + if(config->url_get) { /* there's a node here, if it already is filled-in continue to find an "empty" node */ while(config->url_get && (config->url_get->flags & GETOUT_URL)) @@ -1687,7 +1691,9 @@ ParameterError getparameter(char *flag, /* f or -long-flag */ /* output file */ { struct getout *url; - if(config->url_out || ((config->url_out = config->url_list) != NULL)) { + if(!config->url_out) + config->url_out = config->url_list; + if(config->url_out) { /* there's a node here, if it already is filled-in continue to find an "empty" node */ while(config->url_out && (config->url_out->flags & GETOUT_OUTFILE)) @@ -1824,7 +1830,9 @@ ParameterError getparameter(char *flag, /* f or -long-flag */ /* we are uploading */ { struct getout *url; - if(config->url_out || ((config->url_out = config->url_list) != NULL)) { + if(!config->url_out) + config->url_out = config->url_list; + if(config->url_out) { /* there's a node here, if it already is filled-in continue to find an "empty" node */ while(config->url_out && (config->url_out->flags & GETOUT_UPLOAD)) |