pgsql: Miscellaneous cleanup of regular-expression compiler. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Miscellaneous cleanup of regular-expression compiler.
Date
Msg-id E1ZnB7A-0005iG-A5@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Miscellaneous cleanup of regular-expression compiler.

Revert our previous addition of "all" flags to copyins() and copyouts();
they're no longer needed, and were never anything but an unsightly hack.

Improve a couple of infelicities in the REG_DEBUG code for dumping
the NFA data structure, including adding code to count the total
number of states and arcs.

Add a couple of missed error checks.

Add some more documentation in the README file, and some regression tests
illustrating cases that exceeded the state-count limit and/or took
unreasonable amounts of time before this set of patches.

Back-patch to all supported branches.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/4528f9d69dd2030c8a02218f7e483264627e9ef4

Modified Files
--------------
src/backend/regex/README            |   95 +++++++++++++++++++++++++++++++----
src/backend/regex/regc_nfa.c        |   55 +++++++-------------
src/backend/regex/regcomp.c         |   11 ++--
src/test/regress/expected/regex.out |   35 +++++++++++++
src/test/regress/sql/regex.sql      |   11 ++++
5 files changed, 153 insertions(+), 54 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Miscellaneous cleanup of regular-expression compiler.
Next
From: Tom Lane
Date:
Subject: pgsql: Improve memory-usage accounting in regular-expression compiler.