diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | generic/regcomp.c | 10 |
2 files changed, 15 insertions, 0 deletions
@@ -1,3 +1,8 @@ +2007-11-13 Don Porter <dgp@users.sourceforge.net> + + * generic/regcomp.c: Convert optst() from expensive no-op to a + cheap no-op. + 2007-11-13 Donal K. Fellows <donal.k.fellows@man.ac.uk> * unix/tclUnixChan.c (CreateSocketAddress): Rewrote to use the diff --git a/generic/regcomp.c b/generic/regcomp.c index b9169f9..7faf793 100644 --- a/generic/regcomp.c +++ b/generic/regcomp.c @@ -2103,6 +2103,7 @@ optst( struct vars *v, struct subre *t) { +#if 0 if (t == NULL) { return; } @@ -2117,6 +2118,15 @@ optst( if (t->right != NULL) { optst(v, t->right); } +#else + /* + * DGP (2007-11-13): I assume it was the programmer's intent to + * eventually come back and add code above to optimize subRE trees, + * but the routine coded just spends effort traversing the tree and + * doing nothing. We can do nothing with less effort. + */ + return; +#endif } /* |