[前][次][番号順一覧][スレッド一覧]

ruby-changes:8102

From: akr <ko1@a...>
Date: Mon, 29 Sep 2008 23:39:36 +0900 (JST)
Subject: [ruby-changes:8102] Ruby:r19628 (trunk): * string.c (rb_str_subseq): don't use rb_str_drop_bytes for short

akr	2008-09-29 23:39:20 +0900 (Mon, 29 Sep 2008)

  New Revision: 19628

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=19628

  Log:
    * string.c (rb_str_subseq): don't use rb_str_drop_bytes for short
      string.

  Modified files:
    trunk/ChangeLog
    trunk/string.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 19627)
+++ ChangeLog	(revision 19628)
@@ -1,3 +1,8 @@
+Mon Sep 29 23:32:25 2008  Tanaka Akira  <akr@f...>
+
+	* string.c (rb_str_subseq): don't use rb_str_drop_bytes for short
+	  string.
+
 Mon Sep 29 22:54:29 2008  Tadayoshi Funaba  <tadf@d...>
 
 	* lib/mathn.rb: added String#to_[rc].  moved def_canon.
Index: string.c
===================================================================
--- string.c	(revision 19627)
+++ string.c	(revision 19628)
@@ -1285,8 +1285,9 @@
 {
     VALUE str2;
 
-    if (RSTRING_LEN(str) == beg + len) {
-        str2 = rb_str_new_shared(str);
+    if (RSTRING_LEN(str) == beg + len &&
+        RSTRING_EMBED_LEN_MAX < len) {
+        str2 = rb_str_new_shared(rb_str_new_frozen(str));
         rb_str_drop_bytes(str2, beg);
     }
     else {

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]