aboutsummaryrefslogtreecommitdiffstats
path: root/string.c
diff options
context:
space:
mode:
authorNobuyoshi Nakada <nobu@ruby-lang.org>2019-07-28 07:33:21 +0900
committerNobuyoshi Nakada <nobu@ruby-lang.org>2019-07-28 07:33:21 +0900
commit1d1f98d49c9908f4e3928e582d31fd2e9f252f92 (patch)
treeaf2d132203fb2ce0911f6fbcaeb42755cb7ef1ea /string.c
parentadf13625aeb86961123911cba85c993417a5762a (diff)
downloadruby-1d1f98d49c9908f4e3928e582d31fd2e9f252f92.tar.gz
Reuse match data
* string.c (rb_str_split_m): reuse occupied match data. [Bug #16024]
Diffstat (limited to 'string.c')
-rw-r--r--string.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/string.c b/string.c
index 2805b4d2aa..edcb6aad78 100644
--- a/string.c
+++ b/string.c
@@ -8085,9 +8085,11 @@ rb_str_split_m(int argc, VALUE *argv, VALUE str)
long idx;
int last_null = 0;
struct re_registers *regs;
+ VALUE match = 0;
- while ((end = rb_reg_search(spat, str, start, 0)) >= 0) {
- VALUE match = rb_backref_get();
+ for (; (end = rb_reg_search(spat, str, start, 0)) >= 0;
+ (match ? (rb_match_unbusy(match), rb_backref_set(match)) : (void)0)) {
+ match = rb_backref_get();
if (!result) rb_match_busy(match);
regs = RMATCH_REGS(match);
if (start == end && BEG(0) == END(0)) {
@@ -8120,6 +8122,7 @@ rb_str_split_m(int argc, VALUE *argv, VALUE str)
}
if (!NIL_P(limit) && lim <= ++i) break;
}
+ if (match) rb_match_unbusy(match);
}
if (RSTRING_LEN(str) > 0 && (!NIL_P(limit) || RSTRING_LEN(str) > beg || lim < 0)) {
SPLIT_STR(beg, RSTRING_LEN(str)-beg);