On Jul 10, 2025, at 19:23, David E. Wheeler <[email protected]> wrote:
> Now with the `ISO C90 forbids mixed declarations and code` warning cleared up.
>
> Weird that there’s a failure on Bookworm with Meson [1] (pg_regress diffs
> [2]) but not Bookworm with Configure [3]. Collation issue, perhaps?
David Johnson noticed that this build is 32-bit. I looked at the split_path
function and after trying a couple of things, realized that it was passing an
int8 when the SQL function in Marlena.c passes an int4. This change got the
test passing in my clone (indentation reduced):
```patch
--- a/src/backend/utils/adt/jsonpath_exec.c
+++ b/src/backend/utils/adt/jsonpath_exec.c
@@ -2959,7 +2959,7 @@ executeStringInternalMethod(JsonPathExecContext *cxt,
JsonPathItem *jsp,
C_COLLATION_OID,
CStringGetTextDatum(tmp),
CStringGetTextDatum(from_str),
- DirectFunctionCall1(numeric_int8, NumericGetDatum(n))));
+ DirectFunctionCall1(numeric_int4, NumericGetDatum(n))));
break;
}
default:
```
v12 attached.
Best,
David
v12-0001-Rename-jsonpath-method-arg-tokens.patch
Description: Binary data
v12-0002-Add-additional-jsonpath-string-methods.patch
Description: Binary data
signature.asc
Description: Message signed with OpenPGP
