diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-01-01 05:16:49 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-01-01 05:16:49 +0000 |
commit | 96e7713754c477fe98806b905301356371accb68 (patch) | |
tree | 7cdff8af89fc3d7cf2f22b7329c62f396dc0cb22 | |
parent | 5e22f599825af9fa2134c91932a16815fc1e09e7 (diff) | |
download | ruby-96e7713754c477fe98806b905301356371accb68.tar.gz |
* string.c (rb_str_substr): offset movement bug. a patch from
Vincent Isambart <vincent.isambart at gmail.com> in
[ruby-core:14647]. [ruby-core:14644]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@14825 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | string.c | 4 |
2 files changed, 8 insertions, 2 deletions
@@ -1,3 +1,9 @@ +Tue Jan 1 14:15:04 2008 Yukihiro Matsumoto <matz@ruby-lang.org> + + * string.c (rb_str_substr): offset movement bug. a patch from + Vincent Isambart <vincent.isambart at gmail.com> in + [ruby-core:14647]. [ruby-core:14644] + Tue Jan 1 01:29:04 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> * encoding.c (rb_to_encoding): raises for non-nil, non-encoding, @@ -832,10 +832,10 @@ rb_str_substr(VALUE str, long beg, long len) if (len > -beg) len = -beg; if (-beg * rb_enc_mbmaxlen(enc) < RSTRING_LEN(str) / 8) { beg = -beg; - while (len++ < beg && (e = rb_enc_prev_char(s, e, enc)) != 0); + while (beg-- > len && (e = rb_enc_prev_char(s, e, enc)) != 0); p = e; if (!p) return Qnil; - while (beg-- > 0 && (p = rb_enc_prev_char(s, p, enc)) != 0); + while (len-- > 0 && (p = rb_enc_prev_char(s, p, enc)) != 0); if (!p) return Qnil; len = e - p; goto sub; |