diff options
author | Daniel Stenberg <daniel@haxx.se> | 2023-04-27 16:29:45 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2023-04-27 17:24:47 +0200 |
commit | d567cca1de689fb08da3557f81e7cae34257e438 (patch) | |
tree | 9ecf3b7a85bd5e6d3506ccf8a5ab55e4e3b894ed /lib/mprintf.c | |
parent | 4578ada4a0d14039cac1306fbc736851a9cb753c (diff) | |
download | curl-d567cca1de689fb08da3557f81e7cae34257e438.tar.gz |
checksrc: fix SPACEBEFOREPAREN for conditions starting with "*"
The open paren check wants to warn for spaces before open parenthesis
for if/while/for but also for any function call. In order to avoid
catching function pointer declarations, the logic allows a space if the
first character after the open parenthesis is an asterisk.
I also spotted what we did not include "switch" in the check but we should.
This check is a little lame, but we reduce this problem by not allowing
that space for if/while/for/switch.
Reported-by: Emanuele Torre
Closes #11044
Diffstat (limited to 'lib/mprintf.c')
-rw-r--r-- | lib/mprintf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/mprintf.c b/lib/mprintf.c index 5de935b1f..8e830438d 100644 --- a/lib/mprintf.c +++ b/lib/mprintf.c @@ -400,7 +400,7 @@ static int dprintf_Pass1(const char *format, struct va_stack *vto, /* out of allowed range */ return 1; - switch (*fmt) { + switch(*fmt) { case 'S': flags |= FLAGS_ALT; /* FALLTHROUGH */ |