Merge pull request #6092 from SimonThalvorsen/master

olehermanse · web-flow · commit d5ea5c2edebd · 2026-04-24T13:30:38.000+02:00
Remove maxArg from syntax-description in cases where it is unbound
diff --git a/libpromises/syntax.c b/libpromises/syntax.c
@@ -1293,9 +1293,12 @@ static JsonElement *FnCallTypeToJson(const FnCallType *fn_syntax)
         JsonObjectAppendArray(json_fn, "parameters", params);
     }
     
-    if (!(fn_syntax->argc.min == -1 && fn_syntax->argc.max == -1))
+    if (fn_syntax->argc.min != -1)
     {
         JsonObjectAppendInteger(json_fn, "minArgs", fn_syntax->argc.min);
+    }
+    if (fn_syntax->argc.max != -1)
+    {
         JsonObjectAppendInteger(json_fn, "maxArgs", fn_syntax->argc.max);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1293,9 +1293,12 @@ static JsonElement FnCallTypeToJson(const FnCallType fn_syntax)`
`1293`	`1293`	`JsonObjectAppendArray(json_fn, "parameters", params);`
`1294`	`1294`	`}`
`1295`	`1295`
`1296`		`- if (!(fn_syntax->argc.min == -1 && fn_syntax->argc.max == -1))`
	`1296`	`+ if (fn_syntax->argc.min != -1)`
`1297`	`1297`	`{`
`1298`	`1298`	`JsonObjectAppendInteger(json_fn, "minArgs", fn_syntax->argc.min);`
	`1299`	`+ }`
	`1300`	`+ if (fn_syntax->argc.max != -1)`
	`1301`	`+ {`
`1299`	`1302`	`JsonObjectAppendInteger(json_fn, "maxArgs", fn_syntax->argc.max);`
`1300`	`1303`	`}`
`1301`	`1304`