]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
tool_getparam: avoid `-Wcomma` with Apple clang in C89 mode
authorViktor Szakats <commit@vsz.me>
Mon, 19 Jan 2026 22:21:30 +0000 (23:21 +0100)
committerViktor Szakats <commit@vsz.me>
Mon, 19 Jan 2026 23:20:43 +0000 (00:20 +0100)
Seen with Apple clang 17:
```
curl/src/tool_getparam.c:3062:12: warning: possible misuse of comma operator here [-Wcomma]
 3062 |   for(i = 1, stillflags = TRUE; i < argc && !result; i++) {
      |            ^
curl/src/tool_getparam.c:3062:7: note: cast expression to void to silence warning
 3062 |   for(i = 1, stillflags = TRUE; i < argc && !result; i++) {
      |       ^~~~~
      |       (void)( )
```

Closes #20362

src/tool_getparam.c

index 56f4d58f7338a686a926cd7e96410815110530cd..54b209869dbc41f77f376d038cbc6bcf40ebccab 100644 (file)
@@ -3059,7 +3059,8 @@ ParameterError parse_args(int argc, argv_item_t argv[])
   ParameterError result = PARAM_OK;
   struct OperationConfig *config = global->first;
 
-  for(i = 1, stillflags = TRUE; i < argc && !result; i++) {
+  stillflags = TRUE;
+  for(i = 1; i < argc && !result; i++) {
     orig_opt = convert_tchar_to_UTF8(argv[i]);
     if(!orig_opt)
       return PARAM_NO_MEM;