Re: Remaining dependency on setlocale()
Jeff Davis <pgsql@j-davis.com>
Commits
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
fuzzystrmatch: use pg_ascii_toupper().
- b96a9fd76f32 19 (unreleased) landed
-
Avoid global LC_CTYPE dependency in pg_locale_icu.c.
- 0a90df58cf38 19 (unreleased) landed
-
downcase_identifier(): use method table from locale provider.
- 87b2968df0f8 19 (unreleased) landed
-
ltree: fix case-insensitive matching.
- 806555e3000d 18.2 landed
- 7f007e4a044a 19 (unreleased) landed
-
Fix multibyte issue in ltree_strncasecmp().
- 898991966bc9 14.21 landed
- 335b2f30b468 15.16 landed
- b80227c0a54c 16.12 landed
- b8cfe9dc2e7f 17.8 landed
- f79e239e0bc6 18.2 landed
- 84d5efa7e3eb 19 (unreleased) landed
-
Use multibyte-aware extraction of pattern prefixes.
- 9c8de1596912 19 (unreleased) landed
-
Add pg_iswcased().
- 630706ced04e 19 (unreleased) landed
-
Remove char_tolower() API.
- 1e493158d3d2 19 (unreleased) landed
-
Make regex "max_chr" depend on encoding, not provider.
- 19b966243c38 19 (unreleased) landed
-
Change some callers to use pg_ascii_toupper().
- 99cd8890beca 19 (unreleased) landed
-
Allow pg_locale_t APIs to work when ctype_is_c.
- 147602822597 19 (unreleased) landed
-
Add #define for UNICODE_CASEMAP_BUFSZ.
- 8d299052fe58 19 (unreleased) landed
-
Inline pg_ascii_tolower() and pg_ascii_toupper().
- ec4997a9d733 19 (unreleased) landed
-
Avoid global LC_CTYPE dependency in pg_locale_libc.c.
- f81bf78ce12b 19 (unreleased) landed
-
Force LC_COLLATE to C in postmaster.
- 5e6e42e44fe1 19 (unreleased) landed
-
Change wchar2char() and char2wchar() to accept a locale_t.
- 53cd0b71ee2e 19 (unreleased) landed
-
Use pg_ascii_tolower()/pg_ascii_toupper() where appropriate.
- d81dcc8d6243 19 (unreleased) landed
-
inet_net_pton.c: use pg_ascii_tolower() rather than tolower().
- 8898082a5d3e 18.0 landed
-
isn.c: use pg_ascii_toupper() instead of toupper().
- 7a6880fadc17 18.0 landed
-
contrib/spi/refint.c: use pg_ascii_tolower() instead.
- 78bd364ee39c 18.0 landed
-
copyfromparse.c: use pg_ascii_tolower() rather than tolower().
- 4c787a24e7e2 18.0 landed
-
Revert "Tidy up locale thread safety in ECPG library."
- 3c8e463b0d88 18.0 cited
-
Tidy up locale thread safety in ECPG library.
- 8e993bff5326 18.0 cited
-
All supported systems have locale_t.
- 8d9a9f034e92 17.0 cited
Attachments
- v3-0001-Hold-datcollate-datctype-in-global_libc_locale.patch (text/x-patch) patch v3-0001
- v3-0002-fuzzystrmatch-use-global_libc_locale.patch (text/x-patch) patch v3-0002
- v3-0003-ltree-use-global_libc_locale.patch (text/x-patch) patch v3-0003
- v3-0004-Use-global_libc_locale-for-downcase_identifier-an.patch (text/x-patch) patch v3-0004
- v3-0005-Change-wchar2char-and-char2wchar-to-accept-a-loca.patch (text/x-patch) patch v3-0005
- v3-0006-tsearch-use-global_libc_locale.patch (text/x-patch) patch v3-0006
- v3-0007-Force-LC_COLLATE-to-C-in-postmaster.patch (text/x-patch) patch v3-0007
On Wed, 2025-06-11 at 12:15 -0700, Jeff Davis wrote: > I changed this to a global_libc_locale that includes both LC_COLLATE > and LC_CTYPE (from datcollate and datctype), in case an extension is > relying on strcoll for some reason. .. > This patch series, at least so far, is designed to have zero behavior > changes. Anything with a potential for a behavior change should be a > separate commit, so that if we need to revert it, we can revert the > behavior change without reintroducing a setlocale() dependency. ... > > I reworked it to be less confusing by changing wchar2char/char2wchar > to > take a locale_t instead of pg_locale_t. Hopefully it's an > improvement. ... > > Changed it so that it only sets LC_COLLATE to C, and leaves LC_CTYPE > set to datctype. Attached rebased v3. Regards, Jeff Davis