Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part

David E. Wheeler <david@justatheory.com>

From: "David E. Wheeler" <david@justatheory.com>

To: Florents Tselai <florents.tselai@gmail.com>

Cc: Tom Lane <tgl@sss.pgh.pa.us>, Peter Eisentraut <peter@eisentraut.org>, Robert Haas <robertmhaas@gmail.com>, Alexander Korotkov <aekorotkov@gmail.com>, pgsql-hackers <pgsql-hackers@lists.postgresql.org>, Andrew Dunstan <andrew@dunslane.net>

Date: 2025-07-10T23:23:57Z

Lists: pgsql-hackers

Attachments

v11-0001-Rename-jsonpath-method-arg-tokens.patch (application/octet-stream)
v11-0002-Add-additional-jsonpath-string-methods.patch (application/octet-stream)
(unnamed) (text/plain)

On Jul 10, 2025, at 18:40, David E. Wheeler <david@justatheory.com> wrote:

> Resolved in the attached, which now passes `make check-world` for me.
> 
> Also available as a pull request[1].

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?

Best,

David

[1]: https://cirrus-ci.com/task/5363472541679616
[2]: https://api.cirrus-ci.com/v1/artifact/task/5363472541679616/testrun/build-32/testrun/regress/regress/regression.diffs
[3]: https://cirrus-ci.com/task/5926422495100928