diff options
| author | Marcel Raad <Marcel.Raad@teamviewer.com> | 2019-05-11 14:51:24 +0200 |
|---|---|---|
| committer | Marcel Raad <Marcel.Raad@teamviewer.com> | 2019-05-12 09:31:09 +0200 |
| commit | 6b3dde7fe62ea5a557fd1fd323fac2bcd0c2e9be (patch) | |
| tree | c9bf3ba8868f5d8c8db313cd1cd462dff6a42239 /tests/libtest/lib1156.c | |
| parent | 5f8b9fe81d9681033babb4fbcf983304aa7d11c8 (diff) | |
| download | curl-6b3dde7fe62ea5a557fd1fd323fac2bcd0c2e9be.tar.gz | |
build: fix "clarify calculation precedence" warnings
Codacy/CppCheck warns about this. Consistently use parentheses as we
already do in some places to silence the warning.
Closes https://github.com/curl/curl/pull/3866
Diffstat (limited to 'tests/libtest/lib1156.c')
| -rw-r--r-- | tests/libtest/lib1156.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/libtest/lib1156.c b/tests/libtest/lib1156.c index cb8878c21..f4385b26a 100644 --- a/tests/libtest/lib1156.c +++ b/tests/libtest/lib1156.c @@ -88,7 +88,7 @@ static int onetest(CURL *curl, const char *url, const testparams *p) unsigned int replyselector; char urlbuf[256]; - replyselector = p->flags & F_CONTENTRANGE? 1: 0; + replyselector = (p->flags & F_CONTENTRANGE)? 1: 0; if(p->flags & F_HTTP416) replyselector += 2; msnprintf(urlbuf, sizeof(urlbuf), "%s%04u", url, replyselector); @@ -96,7 +96,7 @@ static int onetest(CURL *curl, const char *url, const testparams *p) test_setopt(curl, CURLOPT_RESUME_FROM, (p->flags & F_RESUME)? 3: 0); test_setopt(curl, CURLOPT_RANGE, !(p->flags & F_RESUME)? "3-1000000": (char *) NULL); - test_setopt(curl, CURLOPT_FAILONERROR, p->flags & F_FAIL? 1: 0); + test_setopt(curl, CURLOPT_FAILONERROR, (p->flags & F_FAIL)? 1: 0); hasbody = 0; res = curl_easy_perform(curl); if(res != p->result) { |