diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2020-11-16 15:16:39 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2020-11-16 15:16:39 -0500 |
| commit | 89aa30a179686d86aeccddf715f039d1a15d2b30 (patch) | |
| tree | eb788a6e1a363c6c967d2bd87fc7d77bddbf534a /src | |
| parent | bea246117b402e59340c9efa0bdbdbe7c1ea8554 (diff) | |
| download | postgresql-89aa30a179686d86aeccddf715f039d1a15d2b30.tar.gz postgresql-89aa30a179686d86aeccddf715f039d1a15d2b30.zip | |
Use "true" not "TRUE" in one ICU function call.
This was evidently missed in commit 6337865f3, which generally did
s/TRUE/true/ everywhere. It escaped notice up to now because ICU
versions before ICU 68 provided definitions of "TRUE" and "FALSE"
regardless. With ICU 68, it fails to compile.
Per report from Condor. Back-patch to v11 where 6337865f3 came in.
(I've not tested v10, where this call originated, but I imagine
it's fine since we defined TRUE in c.h back then.)
Discussion: https://postgr.es/m/7a6f3336165bfe3ca66abcda7966f9d0@stz-bg.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/commands/collationcmds.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/commands/collationcmds.c b/src/backend/commands/collationcmds.c index 8fb51e8c3d1..7fd7f0fdf81 100644 --- a/src/backend/commands/collationcmds.c +++ b/src/backend/commands/collationcmds.c @@ -444,7 +444,7 @@ get_icu_language_tag(const char *localename) UErrorCode status; status = U_ZERO_ERROR; - uloc_toLanguageTag(localename, buf, sizeof(buf), TRUE, &status); + uloc_toLanguageTag(localename, buf, sizeof(buf), true, &status); if (U_FAILURE(status)) ereport(ERROR, (errmsg("could not convert locale name \"%s\" to language tag: %s", |