diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-03-15 13:19:57 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-03-15 13:19:57 -0400 |
commit | b4ce97aa5f603e039ed40af6a208c6f585d1ba9e (patch) | |
tree | 2e0fcdb0862b39802b98684bb43f8deabad44906 /src/backend/executor/execIndexing.c | |
parent | 1bee54e5143f4925297954690c56920d720b7078 (diff) | |
download | postgresql-b4ce97aa5f603e039ed40af6a208c6f585d1ba9e.tar.gz postgresql-b4ce97aa5f603e039ed40af6a208c6f585d1ba9e.zip |
Cope if platform declares mbstowcs_l(), but not locale_t, in <xlocale.h>.
Previously, we included <xlocale.h> only if necessary to get the definition
of type locale_t. According to notes in PGAC_TYPE_LOCALE_T, this is
important because on some versions of glibc that file supplies an
incompatible declaration of locale_t. (This info may be obsolete, because
on my RHEL6 box that seems to be the *only* definition of locale_t; but
there may still be glibc's in the wild for which it's a live concern.)
It turns out though that on FreeBSD and maybe other BSDen, you can get
locale_t from stdlib.h or locale.h but mbstowcs_l() and friends only from
<xlocale.h>. This was leaving us compiling calls to mbstowcs_l() and
friends with no visible prototype, which causes a warning and could
possibly cause actual trouble, since it's not declared to return int.
Hence, adjust the configure checks so that we'll include <xlocale.h>
either if it's necessary to get type locale_t or if it's necessary to
get a declaration of mbstowcs_l().
Report and patch by Aleksander Alekseev, somewhat whacked around by me.
Back-patch to all supported branches, since we have been using
mbstowcs_l() since 9.1.
Diffstat (limited to 'src/backend/executor/execIndexing.c')
0 files changed, 0 insertions, 0 deletions