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 /docs/examples/rtsp.c | |
parent | b228d2952b6762b5c9b851fba0cf391e80c6761a (diff) | |
download | curl-1c3e8bbfedcd3822aeb1bab22fb56c5ecff4295b.tar.gz |
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 'docs/examples/rtsp.c')
-rw-r--r-- | docs/examples/rtsp.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/docs/examples/rtsp.c b/docs/examples/rtsp.c index bdab39542..5c66aa6e5 100644 --- a/docs/examples/rtsp.c +++ b/docs/examples/rtsp.c @@ -61,13 +61,15 @@ static int _getch(void) #define VERSION_STR "V1.0" /* error handling macros */ -#define my_curl_easy_setopt(A, B, C) \ - if((res = curl_easy_setopt((A), (B), (C))) != CURLE_OK) \ +#define my_curl_easy_setopt(A, B, C) \ + res = curl_easy_setopt((A), (B), (C)); \ + if(!res) \ fprintf(stderr, "curl_easy_setopt(%s, %s, %s) failed: %d\n", \ #A, #B, #C, res); -#define my_curl_easy_perform(A) \ - if((res = curl_easy_perform((A))) != CURLE_OK) \ +#define my_curl_easy_perform(A) \ + res = curl_easy_perform(A); \ + if(!res) \ fprintf(stderr, "curl_easy_perform(%s) failed: %d\n", #A, res); |