Skip to content

Commit 90bfae9

Browse files
committed
Update ICU C++ API symbols
Recent ICU versions have added U_SHOW_CPLUSPLUS_HEADER_API, and we need to set this to zero as well to hide the ICU C++ APIs from pg_locale.h Per discussion, we want cpluspluscheck to work cleanly in backbranches, so backpatch both this and its predecessor commit ed26c4e to all supported versions. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/1115793.1754414782%40sss.pgh.pa.us Backpatch-through: 13
1 parent 1beda2c commit 90bfae9

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/include/utils/pg_locale.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@
1818
/* only include the C APIs, to avoid errors in cpluspluscheck */
1919
#undef U_SHOW_CPLUSPLUS_API
2020
#define U_SHOW_CPLUSPLUS_API 0
21+
#undef U_SHOW_CPLUSPLUS_HEADER_API
22+
#define U_SHOW_CPLUSPLUS_HEADER_API 0
2123
#include <unicode/ucol.h>
2224
#endif
2325

0 commit comments

Comments
 (0)