Do not return NULL from pg_newlocale_from_collation().
authorJeff Davis <jdavis@postgresql.org>
Mon, 29 Jul 2024 22:15:11 +0000 (15:15 -0700)
committerJeff Davis <jdavis@postgresql.org>
Mon, 29 Jul 2024 22:18:06 +0000 (15:18 -0700)
commit8240401437c8e261b4ae95fcc4183db98339cc9e
tree013a848688d4c0aa21b69c308237e9125413cf6d
parent6a1d8cef46e654bdc645e9c32412f707471aff2a
Do not return NULL from pg_newlocale_from_collation().

Previously, pg_newlocale_from_collation() returned NULL as a special
case for the DEFAULT_COLLATION_OID if the provider was libc. In that
case the behavior would depend on the last call to setlocale().

Now, consistent with the other providers, it will return a pointer to
default_locale, which is not dependent on setlocale().

Note: for the C and POSIX locales, the locale_t structure within the
pg_locale_t will still be zero, because those locales are implemented
with internal logic and do not use libc at all.

lc_collate_is_c() and lc_ctype_is_c() still depend on setlocale() to
determine the current locale, which will be removed in a subsequent
commit.

Discussion: https://postgr.es/m/cfd9eb85-c52a-4ec9-a90e-a5e4de56e57d@eisentraut.org
Reviewed-by: Peter Eisentraut, Andreas Karlsson
src/backend/utils/adt/pg_locale.c