[ruby-list:42907] Re: apt-listbugs: soap4r での悩み

なかだです。

At Wed, 18 Oct 2006 13:17:44 +0900,
Yukihiro Matsumoto wrote in [ruby-list:42906]:
> |> 置いていただいたSOAPレスポンスを以下のクライアントにかけたところ、手元で
> |> はVIRT 75MBでした（FC5 + ruby 1.8.5 (2006-10-16) [i686-linux]）。
> |
> |10/16バージョンだと、最近のArray#shiftの変更で改善されている可能性もあ
> |りますね。
>
> それは1.9だけでは。しかも、まだバグが残ってるという。

あのばかでかい変更ではなくて、こっちです。

revision 1.137.2.35
date: 2006/10/01 14:45:38;  author: matz;  state: Exp;  lines: +3 -3
* array.c (rb_ary_shift): should not move memory region if array
  body is shared.  a patch from Kent Sibilev <ksruby at gmail.com>.
  [ruby-core:08922]
----------------------------
revision 1.137.2.34
date: 2006/09/24 23:16:53;  author: matz;  state: Exp;  lines: +12 -4
* array.c (rb_ary_shift): should clear shifting top element.
  [ruby-talk:216055]

* array.c (rb_ary_shift): avoid creating shared object if array
  size is small.

しかし、今見返して気づいたんですが、一個余分にコピーしてませんか。


Index: array.c
===================================================================
RCS file: /pub/cvs/ruby/array.c,v
retrieving revision 1.137.2.35
diff -U 2 -p -u -r1.137.2.35 array.c
--- array.c	1 Oct 2006 14:45:38 -0000	1.137.2.35
+++ array.c	18 Oct 2006 06:53:45 -0000
@@ -503,5 +503,5 @@ rb_ary_shift(ary)
     top = RARRAY(ary)->ptr[0];
     if (RARRAY_LEN(ary) < ARY_DEFAULT_SIZE && !FL_TEST(ary, ELTS_SHARED)) {
-	MEMMOVE(RARRAY_PTR(ary), RARRAY_PTR(ary)+1, VALUE, RARRAY_LEN(ary));
+	MEMMOVE(RARRAY_PTR(ary), RARRAY_PTR(ary)+1, VALUE, RARRAY_LEN(ary)-1);
     }
     else {


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