summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-06-16 03:17:54 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-06-16 03:17:54 +0000
commit1cbc622ea7a19acb880e21cf93052791289fd192 (patch)
tree9b29afb3f536df5abc2a9efa4edc07bf7a093e35
parentcc9f1e919518edbee41d602ce215175f52f8f5f5 (diff)
string.c: adjust buffer size
* string.c (tr_trans): adjust buffer size by processed and rest lengths, instead of doubling repeatedly. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@55428 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog5
-rw-r--r--string.c10
2 files changed, 9 insertions, 6 deletions
diff --git a/ChangeLog b/ChangeLog
index 869fd9535e..3b264b63e2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Thu Jun 16 12:17:52 2016 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * string.c (tr_trans): adjust buffer size by processed and rest
+ lengths, instead of doubling repeatedly.
+
Thu Jun 16 11:15:25 2016 Nobuyoshi Nakada <nobu@ruby-lang.org>
* string.c (tr_trans): consider terminator length and fix heap
diff --git a/string.c b/string.c
index 7cb6af2727..73e742696a 100644
--- a/string.c
+++ b/string.c
@@ -6342,9 +6342,8 @@ tr_trans(VALUE str, VALUE src, VALUE repl, int sflag)
c = c0;
if (enc != e1) may_modify = 1;
}
- while (t - buf + tlen >= max) {
- offset = t - buf;
- max *= 2;
+ if ((offset = t - buf) + tlen > max) {
+ max = offset + tlen + (send - s);
REALLOC_N(buf, char, max + termlen);
t = buf + offset;
}
@@ -6415,9 +6414,8 @@ tr_trans(VALUE str, VALUE src, VALUE repl, int sflag)
c = c0;
if (enc != e1) may_modify = 1;
}
- while (t - buf + tlen >= max) {
- offset = t - buf;
- max *= 2;
+ if ((offset = t - buf) + tlen > max) {
+ max = offset + tlen + (long)((send - s) * 1.2);
REALLOC_N(buf, char, max + termlen);
t = buf + offset;
}