diff options
| author | andy <andrew.m.goth@gmail.com> | 2016-11-25 07:31:46 (GMT) |
|---|---|---|
| committer | andy <andrew.m.goth@gmail.com> | 2016-11-25 07:31:46 (GMT) |
| commit | 1000af34faa34a6becf22244a8d7a91bd73f7c45 (patch) | |
| tree | 490d820debff6d8b19ebf40ac93bc4515ccefd59 /generic/tclRegexp.c | |
| parent | d37425bc053928c2dc1de59a8e91638ed468dc9e (diff) | |
| download | tcl-1000af34faa34a6becf22244a8d7a91bd73f7c45.zip tcl-1000af34faa34a6becf22244a8d7a91bd73f7c45.tar.gz tcl-1000af34faa34a6becf22244a8d7a91bd73f7c45.tar.bz2 | |
Allow [array names -regexp] to use backreferences. This capability was broken by [71270e9141]. See also bug [1366683].
Diffstat (limited to 'generic/tclRegexp.c')
| -rw-r--r-- | generic/tclRegexp.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/generic/tclRegexp.c b/generic/tclRegexp.c index ea25d4b..eb23f72 100644 --- a/generic/tclRegexp.c +++ b/generic/tclRegexp.c @@ -502,9 +502,16 @@ Tcl_RegExpMatchObj( { Tcl_RegExp re; - re = Tcl_GetRegExpFromObj(interp, patternObj, - TCL_REG_ADVANCED | TCL_REG_NOSUB); - if (re == NULL) { + /* + * For performance reasons, first try compiling the RE without support for + * subexpressions. On failure, try again without TCL_REG_NOSUB in case the + * RE has backreferences in it. Closely related to [Bug 1366683]. If this + * still fails, an error message will be left in the interpreter. + */ + + if (!(re = Tcl_GetRegExpFromObj(interp, patternObj, + TCL_REG_ADVANCED | TCL_REG_NOSUB)) + && !(re = Tcl_GetRegExpFromObj(interp, patternObj, TCL_REG_ADVANCED))) { return -1; } return Tcl_RegExpExecObj(interp, re, textObj, 0 /* offset */, |