Re: GnomeCanvas (Re: [ruby-gnome2-devel-ja] Maintainer wanted)

[Masao M. <mu...@hi...>]

むとうです。

On Mon, 22 Nov 2004 02:55:43 +0900
KUBO Takehiro <ku...@ji...> wrote:

> 久保です。
> 
> Masao Mutoh <mu...@hi...> writes:
> 
> >> GnomeCanvas が削除された後に GnomeCanvasItem が削除されて、
> >> 
> >>   - gnome_canvas_item_dispose()
> >>   - redraw_if_visible()
> >>   - gnome_canvas_request_redraw()
> >>      --> GnomeCanvas は削除されているのでwarning
> >> 
> >> となっているところまではつきとめました。
> >> とりあえずは rbgnome-canvas-item.c の citem_intialize() で
> >>     g_object_ref(group);
> >> の代りに
> >>     g_object_ref(_SELF(self));
> >> とするとwarningは消えるのですが、これは明らかに間違った対処方法(二重ref)
> >> なのでCVSに入れるわけにはいかない。
> >
> > 他のやつもこの辺が問題だったんですよね。
> >
> > この現象ってrequest_redrawする前段階に二重にunrefされてるから
> > GnomeCanvasが削除されてるんですよね。
> 
> いいえ。GnomeCanvas が削除されること自体は問題ないのですが、その時点で
> GnomeCanvasItem が ruby 側から ref されているのが問題なのではないかと
> 思います。おそらく GnomeCanvasItem は GnomeCanvas のみから参照されてい
> るという前提ではないかと。
> 試しに ruby 側からの ref をなくすために gnomecanvas/sample/canvas.rb
> に以下のパッチを当てて実行したところ、終了時のエラーが出なくなりました。
> # signal_connect を無効化させているのは、signal_connect は ruby オブジェ
> # クトへの参照を保持している場合があるためです。
> 
> citem_intialize() の g_object_ref(group) を消した状態でも同じです。

> > だとすれば、二重refであっても、それが適正な回数であれば問題ないと思います。
> > 問題は適正かどうかなのですが、これはこれで難しそうですね。
> > この場合のメモリリークは許容範囲内かもしれないですし(^^;)。
> 
> glib のソースの g_object_last_unref() に手を加えて個々のオブジェクトが
> 削除されるタイミングを見たところ、citem_intialize() に
> g_object_ref(group) がついている状態だと GnomeCanvasGroup は最後まで削
> 除されませんでした。おそらくむとうさんのところでエラーが消えたのは
> libgnomecanvas 側、ruby 側両方の終了処理が終わっても GnomeCanvasGroup
> の ref が残っていて最後まで gnome_canvas_item_dispose() が呼ばれなかっ
> たためではないかと思います。
> 
> > もしかしたら他のメソッドでg_object_refをしないといけない場所があるのかも
> > しれませんね。

> Ruby/GLib2 に手をいれないといけないような予感が....。
> 今の Ruby/GLib2 だと、GObject に対応する ruby object を生成すると
> GObject の refcount が 1 上がるのですが、refcount を上げずに weakref
> のみで参照する ruby object が必要かも。

なるほど...。
あ、もしかして、RBGTK_INITIALIZEでg_object_refしてるのもマズイですかねぇ。
一度、そのあたりを見直す時期なのかもしれませんね。
 
> > ちなみに、これって、久保さんの環境だとサンプルでも再現しますか？
> > というのは私の環境ではもうこの現象出ていないんですよね。
> 
> 再現しています。

うーん、なんでだろう。やっぱり何かのバージョンの違いなのかなぁ。
 
> あと、むとうさんの今日の commit で、エラー時の core dump がまた起きる
> ようになりました。finalizer が走っている最中に例外が上がると core を吐
> くようです。

げげっ、すみません。直します。うぬぬぬ。

-- 
.:% Masao Mutoh<mu...@hi...>