summaryrefslogtreecommitdiffstats
path: root/generic/regguts.h
diff options
context:
space:
mode:
authordgp <dgp@users.sourceforge.net>2015-10-21 13:50:55 (GMT)
committerdgp <dgp@users.sourceforge.net>2015-10-21 13:50:55 (GMT)
commit4e08431e8790a66cea86d33c8c770626396509c0 (patch)
tree7339a003ab56cd793a40da29f03f7f18a52a80bb /generic/regguts.h
parentb22ebcce6b9f2e861a4c9fed13421df0c76769d6 (diff)
parent16ccc54387b39505723e3413eb4d755ea7cdb3ff (diff)
downloadtcl-4e08431e8790a66cea86d33c8c770626396509c0.zip
tcl-4e08431e8790a66cea86d33c8c770626396509c0.tar.gz
tcl-4e08431e8790a66cea86d33c8c770626396509c0.tar.bz2
[1080042][8f245009b0] Big bundle of regexp engine fixes and improvements
contributed from Tom Lane of the postgres project.
Diffstat (limited to 'generic/regguts.h')
-rw-r--r--generic/regguts.h20
1 files changed, 12 insertions, 8 deletions
diff --git a/generic/regguts.h b/generic/regguts.h
index ac625d5..72bdcb3 100644
--- a/generic/regguts.h
+++ b/generic/regguts.h
@@ -265,8 +265,10 @@ struct arc {
struct state *from; /* where it's from (and contained within) */
struct state *to; /* where it's to */
struct arc *outchain; /* *from's outs chain or free chain */
-#define freechain outchain
+ struct arc *outchainRev; /* back-link in *from's outs chain */
+#define freechain outchain /* we do not maintain "freechainRev" */
struct arc *inchain; /* *to's ins chain */
+ struct arc *inchainRev; /* back-link in *to's ins chain */
struct arc *colorchain; /* color's arc chain */
struct arc *colorchainRev; /* back-link in color's arc chain */
};
@@ -305,9 +307,6 @@ struct nfa {
struct colormap *cm; /* the color map */
color bos[2]; /* colors, if any, assigned to BOS and BOL */
color eos[2]; /* colors, if any, assigned to EOS and EOL */
- size_t size; /* Current NFA size; differs from nstates as
- * it also counts the number of states created
- * by children of this state. */
struct vars *v; /* simplifies compile error reporting */
struct nfa *parent; /* parent NFA, if any */
};
@@ -337,11 +336,16 @@ struct cnfa {
#define NULLCNFA(cnfa) ((cnfa).nstates == 0)
/*
- * Used to limit the maximum NFA size to something sane. [Bug 1810264]
+ * This symbol limits the transient heap space used by the regex compiler,
+ * and thereby also the maximum complexity of NFAs that we'll deal with.
+ * Currently we only count NFA states and arcs against this; the other
+ * transient data is generally not large enough to notice compared to those.
+ * Note that we do not charge anything for the final output data structures
+ * (the compacted NFA and the colormap).
*/
-
-#ifndef REG_MAX_STATES
-# define REG_MAX_STATES 100000
+#ifndef REG_MAX_COMPILE_SPACE
+#define REG_MAX_COMPILE_SPACE \
+ (100000 * sizeof(struct state) + 100000 * sizeof(struct arcbatch))
#endif
/*