lib: silence warnings on comma misuse

Building curl with -Wcomma, I see warnings about "possible misuse of
comma operator here" and moving fields assignment out of the for() fixes
it.

Closes #13392

diff --git a/lib/cookie.c b/lib/cookie.c
index c1ed2913c40caa090375c11d2b15166b329f0e90..837caaab384466756138df88b886224390176bcd 100644 (file)
--- a/lib/cookie.c
+++ b/lib/cookie.c
@@ -886,7 +886,8 @@ Curl_cookie_add(struct Curl_easy *data,
      * Now loop through the fields and init the struct we already have
      * allocated
      */
-    for(ptr = firstptr, fields = 0; ptr && !badcookie;
+    fields = 0;
+    for(ptr = firstptr; ptr && !badcookie;
         ptr = strtok_r(NULL, "\t", &tok_buf), fields++) {
       switch(fields) {
       case 0:

diff --git a/lib/ldap.c b/lib/ldap.c
index e2546aa59ecacc38c4315e1b4f2ca115689b6548..a545f87d9dcff6305be7ad2cd0f380a38de746bf 100644 (file)
--- a/lib/ldap.c
+++ b/lib/ldap.c
@@ -546,7 +546,8 @@ static CURLcode ldap_do(struct Curl_easy *data, bool *done)
     goto quit;
   }
 
-  for(num = 0, entryIterator = ldap_first_entry(server, ldapmsg);
+  num = 0;
+  for(entryIterator = ldap_first_entry(server, ldapmsg);
       entryIterator;
       entryIterator = ldap_next_entry(server, entryIterator), num++) {
     BerElement *ber = NULL;