| 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 |
| 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? |