diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2016-12-14 01:29:44 +0100 |
|---|---|---|
| committer | ng0 <ng0@infotropique.org> | 2017-08-22 15:10:44 +0000 |
| commit | 1aded807aedb6ef72ea7f2ba53e0daf341468eb8 (patch) | |
| tree | ec6f2bcd8e9d61b799009299c1f461d61e8982b4 /lib/formdata.c | |
| parent | b224d5c53fc22dec5f14d741d7a499b4c4c6a446 (diff) | |
| download | gnurl-1aded807aedb6ef72ea7f2ba53e0daf341468eb8.tar.gz gnurl-1aded807aedb6ef72ea7f2ba53e0daf341468eb8.tar.bz2 gnurl-1aded807aedb6ef72ea7f2ba53e0daf341468eb8.zip | |
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 'lib/formdata.c')
| -rw-r--r-- | lib/formdata.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/formdata.c b/lib/formdata.c index 53dee39f4..abd2da075 100644 --- a/lib/formdata.c +++ b/lib/formdata.c @@ -949,8 +949,8 @@ void Curl_formclean(struct FormData **form_ptr) if(form->type <= FORM_CONTENT) free(form->line); /* free the line */ free(form); /* free the struct */ - - } while((form = next) != NULL); /* continue */ + form = next; + } while(form); /* continue */ *form_ptr = NULL; } @@ -1031,8 +1031,8 @@ void curl_formfree(struct curl_httppost *form) free(form->contenttype); /* free the content type */ free(form->showfilename); /* free the faked file name */ free(form); /* free the struct */ - - } while((form = next) != NULL); /* continue */ + form = next; + } while(form); /* continue */ } #ifndef HAVE_BASENAME @@ -1374,8 +1374,8 @@ CURLcode Curl_getformdata(struct Curl_easy *data, if(result) break; } - - } while((post = post->next) != NULL); /* for each field */ + post = post->next; + } while(post); /* for each field */ /* end-boundary for everything */ if(!result) |