From: Yukihiro Matsumoto <matz@...>
Date: 2004-10-12T15:26:57+09:00
Subject: [ruby-dev:24484] Re: can't build ruby 1.9.0 on mswin32

まつもと ゆきひろです

In message "Re: [ruby-dev:24483] Re: can't build ruby 1.9.0 on mswin32"
    on Tue, 12 Oct 2004 15:22:20 +0900, Yukihiro Matsumoto <matz@ruby-lang.org> writes:

||[ruby-core:3459] メモリリークの件が直ってないのでものすごく
||メモリを食うというのもありますが、ext/の下を考えなしに全部処
||理してるのはまずいですね。
|
|ああ、これpendingリストには入ってるんですが。なにがいけない
|んでしょうねえ。

よく見たら、すぐ後にGuyが指摘していますね。

|Mon Sep 13 19:16:33 2004  WATANABE Hirofumi  <eban@ruby-lang.org>
|
|        * eval.c (blk_copy_prev): need frame_dup().  [ruby-dev:24103]
|
| frame_dup(&tmp->frame) conflict with the first 
| ALLOC_N(VALUE, tmp->frame.argc) (make allocation twice)
|
|Guy Decoux

ということは、以下のパッチで直っちゃうんでしょうか。

--- eval.c	6 Oct 2004 15:15:11 -0000	1.616.2.52
+++ eval.c	12 Oct 2004 06:25:38 -0000
@@ -7792,8 +7792,2 @@ blk_copy_prev(block)
 	MEMCPY(tmp, block->prev, struct BLOCK, 1);
-	if (tmp->frame.argc > 0) {
-	    tmp->frame.argv = ALLOC_N(VALUE, tmp->frame.argc);
-	    MEMCPY(tmp->frame.argv, block->prev->frame.argv, VALUE, tmp->frame.argc);
-	    tmp->frame.flags |= FRAME_MALLOC;
-	}
-	scope_dup(tmp->scope);
 	frame_dup(&tmp->frame);