pgsql: Move is_valid_ascii() to ascii.h. - Mailing list pgsql-committers

From Nathan Bossart
Subject pgsql: Move is_valid_ascii() to ascii.h.
Date
Msg-id E1rUW4z-003ikB-Fa@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Move is_valid_ascii() to ascii.h.

This function requires simd.h, which is a rather large dependency
for a widely-used header file like pg_wchar.h.  Furthermore, there
is a report of a third-party tool that is struggling to use
pg_wchar.h due to its dependence on simd.h (presumably because
simd.h uses several intrinsics).  Moving the function to the much
less popular ascii.h resolves these issues for now.

This commit is back-patched for the benefit of the aforementioned
third-party tool.  The simd.h dependency was only added in v16,
but we've opted to back-patch to v15 so that is_valid_ascii() lives
in the same file for all versions where it exists.  This could
break existing third-party code that uses the function, but we
couldn't find any examples of such code.  It should be possible to
fix any code that this commit breaks by including ascii.h in the
file that uses is_valid_ascii().

Author: Jubilee Young
Reviewed-by: Tom Lane, John Naylor, Andres Freund, Eric Ridge
Discussion: https://postgr.es/m/CAPNHn3oKJJxMsYq%2BqLYzVJOFrUcOr4OF1EC-KtFT-qh8nOOOtQ%40mail.gmail.com
Backpatch-through: 15

Branch
------
REL_15_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/3726c1cb0e3bd18d8916978e18d95251d07360c3

Modified Files
--------------
src/common/wchar.c        |  1 +
src/include/mb/pg_wchar.h | 53 -----------------------------------------------
src/include/utils/ascii.h | 52 ++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 53 insertions(+), 53 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix incompatibilities with libxml2 >= 2.12.0.
Next
From: David Rowley
Date:
Subject: pgsql: Doc: mention foreign keys can reference unique indexes