From: Hiroshi IGARASHI <igarashi@...>
Date: 2000-02-23T04:11:20+09:00
Subject: [ruby-list:20950] Re: Ruby/Gtk の CList にパッチ

いがらしです。返答が遅くなりすみません。

Ruby/GTKに関するパッチそのものは、ruby-extか僕個人当てにお願いします。

At Tue, 15 Feb 2000 22:16:59 +0900,
in [ruby-list:20737] Ruby/Gtk の CList にパッチ,
YASUI Kentarow <kenyasui@alles.or.jp> wrote:
> 
> Ruby/GtkのCListで、movetoとrow_is_visibleが使いたかったのですが、まだ対応
> していないようだったのでパッチを作成しました。取り込んでもらえると嬉しい
> です。

取り込みます。

> row_is_visibleはrow_visible?とかのメソッド名の方がRubyらしいかもしれません
> が、Ruby/Gtkでのメソッドのネーミングルールがわからなかったのでそのままに
> してあります。

基本的には

  - Rubyの慣習に従う
  - C/GTKの関数との対応を分かりやすく

で、gtk_clist_row_is_visible は Gtk::CList#row_is_visible? に
なるかな、と思いきや……

gtk_clist_row_is_visible()はtrue/falseではなく
3つの値をとりうる列挙型GtkVisibility

    typedef enum
     {
       GTK_VISIBILITY_NONE,
       GTK_VISIBILITY_PARTIAL,
       GTK_VISIBILITY_FULL
     } GtkVisibility;

を返すので、`?'で終るメソッド名は不適切かもしれません。
とりあえず`?'なしにしておきます。


ただ`?'がついてもつかなくても

    if clist.row_is_visible(row_num) # 常に真とみなされてしまう
       ...
    end

などとうっかり書いてしまいそうです。
(PARTIAL/FULLを意図しているならば)正しくは

    if clist.row_is_visible(row_num) != VISIBILITY_NONE
       ...
    end

ですね。

GtkVisibilityの値をそれぞれ 
    GTK_VISIBILITY_NONE     -> false
    GTK_VISIBILITY_PARTIAL  -> (nil/false以外の何か)
    GTK_VISIBILITY_FULL     -> true
にmappingするというのもありでしょうか？

--
五十嵐  宏  (Hiroshi IGARASHI)