diff options
author | Marcel Raad <Marcel.Raad@teamviewer.com> | 2017-07-16 13:44:54 +0200 |
---|---|---|
committer | Marcel Raad <Marcel.Raad@teamviewer.com> | 2017-07-16 14:02:59 +0200 |
commit | fb3b0f25ef9b20b40e48d2a507968a975ef09bb9 (patch) | |
tree | a471bd461f3e95b5617323e50b67564cad1d2901 /tests/libtest/lib1900.c | |
parent | 0bdb81125008207c9be90c69ec9393061eaab10b (diff) |
libtest: fix MSVC warning C4706
With warning level 4, MSVC warns about assignments within conditional
expressions. Change the while loop to a do-while loop to fix this. This
change is also consistent with CODE_STYLE.md.
Diffstat (limited to 'tests/libtest/lib1900.c')
-rw-r--r-- | tests/libtest/lib1900.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tests/libtest/lib1900.c b/tests/libtest/lib1900.c index b55f3b7d8..cac1dd1d9 100644 --- a/tests/libtest/lib1900.c +++ b/tests/libtest/lib1900.c @@ -189,8 +189,9 @@ int test(char *URL) abort_on_test_timeout(); /* See how the transfers went */ - while((msg = curl_multi_info_read(m, &msgs_left))) { - if(msg->msg == CURLMSG_DONE) { + do { + msg = curl_multi_info_read(m, &msgs_left); + if(msg && msg->msg == CURLMSG_DONE) { int i, found = 0; /* Find out which handle this message is about */ @@ -203,7 +204,7 @@ int test(char *URL) printf("Handle %d Completed with status %d\n", i, msg->data.result); curl_multi_remove_handle(m, handles[i]); } - } + } while(msg); if(handlenum == num_handles && !running) { break; /* done */ |