summaryrefslogtreecommitdiff
path: root/src/tool_urlglob.c
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2023-04-27 16:29:45 +0200
committerDaniel Stenberg <daniel@haxx.se>2023-04-27 17:24:47 +0200
commitd567cca1de689fb08da3557f81e7cae34257e438 (patch)
tree9ecf3b7a85bd5e6d3506ccf8a5ab55e4e3b894ed /src/tool_urlglob.c
parent4578ada4a0d14039cac1306fbc736851a9cb753c (diff)
downloadcurl-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 'src/tool_urlglob.c')
-rw-r--r--src/tool_urlglob.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/tool_urlglob.c b/src/tool_urlglob.c
index 44dd9a748..fe1ce64c6 100644
--- a/src/tool_urlglob.c
+++ b/src/tool_urlglob.c
@@ -100,7 +100,7 @@ static CURLcode glob_set(struct URLGlob *glob, char **patternp,
pat->globindex = globindex;
while(!done) {
- switch (*pattern) {
+ switch(*pattern) {
case '\0': /* URL ended while set was still open */
return GLOBERROR("unmatched brace", opos, CURLE_URL_MALFORMAT);
@@ -411,7 +411,7 @@ static CURLcode glob_parse(struct URLGlob *glob, char *pattern,
res = glob_fixed(glob, glob->glob_buffer, sublen);
}
else {
- switch (*pattern) {
+ switch(*pattern) {
case '\0': /* done */
break;