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

From: "David E(dot) Wheeler" <david(at)justatheory(dot)com>
To: Florents Tselai <florents(dot)tselai(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(at)eisentraut(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>
Subject: Re: PATCH: jsonpath string methods: lower, upper, initcap, l/r/btrim, replace, split_part
Date: 2025-07-10 23:23:57
Message-ID: D3137F93-B6CA-45D0-BF9A-43ED5DF7859B@justatheory.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Jul 10, 2025, at 18:40, David E. Wheeler <david(at)justatheory(dot)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

Attachment Content-Type Size
v11-0001-Rename-jsonpath-method-arg-tokens.patch application/octet-stream 3.9 KB
v11-0002-Add-additional-jsonpath-string-methods.patch application/octet-stream 48.5 KB
unknown_filename text/plain 2 bytes

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2025-07-10 23:47:25 Re: Remaining dependency on setlocale()
Previous Message Peter Geoghegan 2025-07-10 22:58:58 Re: Can can I make an injection point wait occur no more than once?