Skip to content

Commit 6155a96

Browse files
jeff-davispull[bot]
jeff-davis
authored and
pull[bot]
committed
Remove unnecessary #ifdef USE_ICU and branch.
Now that the provider-independent API pg_strnxfrm() is available, we no longer need the special cases for ICU in hashfunc.c and varchar.c. Reviewed-by: Peter Eisentraut, Peter Geoghegan Discussion: https://postgr.es/m/a581136455c940d7bd0ff482d3a2bd51af25a94f.camel%40j-davis.com
1 parent 9b9780f commit 6155a96

File tree

2 files changed

+71
-102
lines changed

2 files changed

+71
-102
lines changed

‎src/backend/access/hash/hashfunc.c

Lines changed: 42 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -289,34 +289,27 @@ hashtext(PG_FUNCTION_ARGS)
289289
}
290290
else
291291
{
292-
#ifdef USE_ICU
293-
if (mylocale->provider == COLLPROVIDER_ICU)
294-
{
295-
Size bsize, rsize;
296-
char *buf;
297-
const char *keydata = VARDATA_ANY(key);
298-
size_t keylen = VARSIZE_ANY_EXHDR(key);
299-
300-
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
301-
buf = palloc(bsize + 1);
302-
303-
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
304-
if (rsize != bsize)
305-
elog(ERROR, "pg_strnxfrm() returned unexpected result");
306-
307-
/*
308-
* In principle, there's no reason to include the terminating NUL
309-
* character in the hash, but it was done before and the behavior
310-
* must be preserved.
311-
*/
312-
result = hash_any((uint8_t *) buf, bsize + 1);
313-
314-
pfree(buf);
315-
}
316-
else
317-
#endif
318-
/* shouldn't happen */
319-
elog(ERROR, "unsupported collprovider: %c", mylocale->provider);
292+
Size bsize, rsize;
293+
char *buf;
294+
const char *keydata = VARDATA_ANY(key);
295+
size_t keylen = VARSIZE_ANY_EXHDR(key);
296+
297+
298+
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
299+
buf = palloc(bsize + 1);
300+
301+
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
302+
if (rsize != bsize)
303+
elog(ERROR, "pg_strnxfrm() returned unexpected result");
304+
305+
/*
306+
* In principle, there's no reason to include the terminating NUL
307+
* character in the hash, but it was done before and the behavior
308+
* must be preserved.
309+
*/
310+
result = hash_any((uint8_t *) buf, bsize + 1);
311+
312+
pfree(buf);
320313
}
321314

322315
/* Avoid leaking memory for toasted inputs */
@@ -350,35 +343,27 @@ hashtextextended(PG_FUNCTION_ARGS)
350343
}
351344
else
352345
{
353-
#ifdef USE_ICU
354-
if (mylocale->provider == COLLPROVIDER_ICU)
355-
{
356-
Size bsize, rsize;
357-
char *buf;
358-
const char *keydata = VARDATA_ANY(key);
359-
size_t keylen = VARSIZE_ANY_EXHDR(key);
360-
361-
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
362-
buf = palloc(bsize + 1);
363-
364-
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
365-
if (rsize != bsize)
366-
elog(ERROR, "pg_strnxfrm() returned unexpected result");
367-
368-
/*
369-
* In principle, there's no reason to include the terminating NUL
370-
* character in the hash, but it was done before and the behavior
371-
* must be preserved.
372-
*/
373-
result = hash_any_extended((uint8_t *) buf, bsize + 1,
374-
PG_GETARG_INT64(1));
375-
376-
pfree(buf);
377-
}
378-
else
379-
#endif
380-
/* shouldn't happen */
381-
elog(ERROR, "unsupported collprovider: %c", mylocale->provider);
346+
Size bsize, rsize;
347+
char *buf;
348+
const char *keydata = VARDATA_ANY(key);
349+
size_t keylen = VARSIZE_ANY_EXHDR(key);
350+
351+
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
352+
buf = palloc(bsize + 1);
353+
354+
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
355+
if (rsize != bsize)
356+
elog(ERROR, "pg_strnxfrm() returned unexpected result");
357+
358+
/*
359+
* In principle, there's no reason to include the terminating NUL
360+
* character in the hash, but it was done before and the behavior
361+
* must be preserved.
362+
*/
363+
result = hash_any_extended((uint8_t *) buf, bsize + 1,
364+
PG_GETARG_INT64(1));
365+
366+
pfree(buf);
382367
}
383368

384369
PG_FREE_IF_COPY(key, 0);

‎src/backend/utils/adt/varchar.c

Lines changed: 29 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -1021,32 +1021,24 @@ hashbpchar(PG_FUNCTION_ARGS)
10211021
}
10221022
else
10231023
{
1024-
#ifdef USE_ICU
1025-
if (mylocale->provider == COLLPROVIDER_ICU)
1026-
{
1027-
Size bsize, rsize;
1028-
char *buf;
1024+
Size bsize, rsize;
1025+
char *buf;
10291026

1030-
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
1031-
buf = palloc(bsize + 1);
1027+
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
1028+
buf = palloc(bsize + 1);
10321029

1033-
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
1034-
if (rsize != bsize)
1035-
elog(ERROR, "pg_strnxfrm() returned unexpected result");
1030+
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
1031+
if (rsize != bsize)
1032+
elog(ERROR, "pg_strnxfrm() returned unexpected result");
10361033

1037-
/*
1038-
* In principle, there's no reason to include the terminating NUL
1039-
* character in the hash, but it was done before and the behavior
1040-
* must be preserved.
1041-
*/
1042-
result = hash_any((uint8_t *) buf, bsize + 1);
1034+
/*
1035+
* In principle, there's no reason to include the terminating NUL
1036+
* character in the hash, but it was done before and the behavior
1037+
* must be preserved.
1038+
*/
1039+
result = hash_any((uint8_t *) buf, bsize + 1);
10431040

1044-
pfree(buf);
1045-
}
1046-
else
1047-
#endif
1048-
/* shouldn't happen */
1049-
elog(ERROR, "unsupported collprovider: %c", mylocale->provider);
1041+
pfree(buf);
10501042
}
10511043

10521044
/* Avoid leaking memory for toasted inputs */
@@ -1084,33 +1076,25 @@ hashbpcharextended(PG_FUNCTION_ARGS)
10841076
}
10851077
else
10861078
{
1087-
#ifdef USE_ICU
1088-
if (mylocale->provider == COLLPROVIDER_ICU)
1089-
{
1090-
Size bsize, rsize;
1091-
char *buf;
1079+
Size bsize, rsize;
1080+
char *buf;
10921081

1093-
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
1094-
buf = palloc(bsize + 1);
1082+
bsize = pg_strnxfrm(NULL, 0, keydata, keylen, mylocale);
1083+
buf = palloc(bsize + 1);
10951084

1096-
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
1097-
if (rsize != bsize)
1098-
elog(ERROR, "pg_strnxfrm() returned unexpected result");
1085+
rsize = pg_strnxfrm(buf, bsize + 1, keydata, keylen, mylocale);
1086+
if (rsize != bsize)
1087+
elog(ERROR, "pg_strnxfrm() returned unexpected result");
10991088

1100-
/*
1101-
* In principle, there's no reason to include the terminating NUL
1102-
* character in the hash, but it was done before and the behavior
1103-
* must be preserved.
1104-
*/
1105-
result = hash_any_extended((uint8_t *) buf, bsize + 1,
1106-
PG_GETARG_INT64(1));
1089+
/*
1090+
* In principle, there's no reason to include the terminating NUL
1091+
* character in the hash, but it was done before and the behavior
1092+
* must be preserved.
1093+
*/
1094+
result = hash_any_extended((uint8_t *) buf, bsize + 1,
1095+
PG_GETARG_INT64(1));
11071096

1108-
pfree(buf);
1109-
}
1110-
else
1111-
#endif
1112-
/* shouldn't happen */
1113-
elog(ERROR, "unsupported collprovider: %c", mylocale->provider);
1097+
pfree(buf);
11141098
}
11151099

11161100
PG_FREE_IF_COPY(key, 0);

0 commit comments

Comments
 (0)