[ruby-dev:33243] Re: [ruby-cvs:22386] Ruby:r15149 (trunk): * string.c (rb_str_each_char): move forward.

なかだです。

At Mon, 21 Jan 2008 15:56:43 +0900,
Nobuyoshi Nakada wrote in [ruby-dev:33240]:
> > 単純に rb_enc_mbclen を呼ぶのではうまくいかないことがあるん
> > でしょうか?
> 
> 変更がないことが確認できればそのほうがいいんですけどね。

freezeしたコピーを作っておいて、変更の影響を受けないようにすると
いうのを思い付きました。当然のことながら格段に速くなるようです。


Index: string.c
===================================================================
--- string.c	(revision 15153)
+++ string.c	(working copy)
@@ -4592,20 +4592,15 @@ rb_str_each_char(VALUE str)
 {
     int i, len, n;
-    const char *ptr, *s;
+    const char *ptr;
     rb_encoding *enc;
 
     RETURN_ENUMERATOR(str, 0, 0);
+    str = rb_str_new4(str);
     ptr = RSTRING_PTR(str);
     len = RSTRING_LEN(str);
     enc = rb_enc_get(str);
-    n = rb_enc_mbclen(ptr, ptr + len, enc);
-    for (i = 0; i < len;) {
+    for (i = 0; i < len; i += len) {
+	n = rb_enc_mbclen(ptr + i, ptr + len, enc);
 	rb_yield(rb_str_subseq(str, i, n));
-	ptr = RSTRING_PTR(str);
-	len = RSTRING_LEN(str);
-	if ((i += n) >= len) break;
-	enc = rb_enc_get(str);
-	s = rb_enc_right_char_head(ptr, ptr + i, enc);
-	n = rb_enc_mbclen(s, ptr + len, enc);
     }
     return str;


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦