This is a multi-part message in MIME format.

--Multipart_Sat__4_Aug_2001_01:15:15_+0900_0825d390
Content-Type: text/plain; charset=ISO-2022-JP
Content-Transfer-Encoding: 7bit

むとうです。

Gtk::Toolbarでiconをnil指定するとエラーになってしまいます。
で、ここでiconの部分をnil指定してもエラーにならないようにしてみました。
#いう使い方する人は少ないとは思いますが...。

toolbar.append_item("File", "File - Tooltips", "File", icon, nil)
toolbar.append_item("File", "File - Tooltips", "File", nil, nil) 
#どちらもOK。

Gtk::Toolbar::prepend_item, Gtk::Toolbar::insert_itemも
同様にnil可にしました。


から、スペルミスがあったのでもなおしました。

CHILD_SPCAE → CHILD_SPACE


問題ないようでしたら反映していただけませんか? > 五鬚気



ところで、ここんとこで2つほど不自然に感じました。

A. のインタフェイスって

 1. toolbar.append_item(text, ttext, ptext, icon, func1) { func2 }

  のようにfuncを2カ把蟲舛任るようになっていますよね;
  #func1の方が優先。
  GTK+と引数を一致させるために上記のようにしていると思うのですが、
  func2のみを指定する場合は

 2. toolbar.append_item(text, ttext, ptext, icon) { func2 }

  のように指定できた方が自然な気がします。
 で、2.のパターンも使えるように気靴燭い里任垢いかがでしょうか;
 #一応、1.も使えるよう(互換性は残したまま)にしたいと思います。


B. 、Gtk::Toolbar::ICONSという定数とGtk::TOOLBAR_ICONSという定数、
   2つの定数が定義されてしまっています(gtk+レベルではともにGTK_TOOLBAR_ICONS)。
  
   別に2つ定義されていても問題ないとは思うのですが、どちらが推奨なのでしょう?
   気持ち的にはGtk::Toolbar::ICONSの方なんですが。
   さらに言えば、Gtk::Toolbar::Constants::ICONSなのかな。
   #この辺っていつか整理するんでしょうか?


では。
----
○● むとう まさお(MUTOH Masao)    
●○ E-Mail:mutoh / highway.ne.jp


--Multipart_Sat__4_Aug_2001_01:15:15_+0900_0825d390
Content-Type: application/octet-stream;
 namebgtktoolbar.c.patch"
Content-Disposition: attachment;
 filenamebgtktoolbar.c.patch"
Content-Transfer-Encoding: base64

LS0tIHJiZ3RrdG9vbGJhci5jLm9yZwlTYXQgQXVnICA0IDAwOjI5OjAzIDIwMDEKKysrIHJiZ3Rr
dG9vbGJhci5jCVNhdCBBdWcgIDQgMDA6MzA6MjUgMjAwMQpAQCAtNDUsNyArNDUsNyBAQAogCQkJ
CSAgTklMX1AodGV4dCk/TlVMTDpTVFIyQ1NUUih0ZXh0KSwKIAkJCQkgIE5JTF9QKHR0ZXh0KT9O
VUxMOlNUUjJDU1RSKHR0ZXh0KSwKIAkJCQkgIE5JTF9QKHB0ZXh0KT9OVUxMOlNUUjJDU1RSKHB0
ZXh0KSwKLQkJCQkgIGdldF93aWRnZXQoaWNvbiksCisJCQkJICBOSUxfUChpY29uKT9OVUxMOmdl
dF93aWRnZXQoaWNvbiksCiAJCQkJICBleGVjX2NhbGxiYWNrLAogCQkJCSAgKGdwb2ludGVyKWZ1
bmMpOwogICAgIHJldHVybiByZXQgPyBtYWtlX3dpZGdldChnV2lkZ2V0LCByZXQpIDogUW5pbDsK
QEAgLTY1LDcgKzY1LDcgQEAKIAkJCQkgICBOSUxfUCh0ZXh0KT9OVUxMOlNUUjJDU1RSKHRleHQp
LAogCQkJCSAgIE5JTF9QKHR0ZXh0KT9OVUxMOlNUUjJDU1RSKHR0ZXh0KSwKIAkJCQkgICBOSUxf
UChwdGV4dCk/TlVMTDpTVFIyQ1NUUihwdGV4dCksCi0JCQkJICAgZ2V0X3dpZGdldChpY29uKSwK
KwkJCQkgICBOSUxfUChpY29uKT9OVUxMOmdldF93aWRnZXQoaWNvbiksCiAJCQkJICAgZXhlY19j
YWxsYmFjaywKIAkJCQkgICAoZ3BvaW50ZXIpZnVuYyk7CiAgICAgcmV0dXJuIHJldCA/IG1ha2Vf
d2lkZ2V0KGdXaWRnZXQsIHJldCkgOiBRbmlsOwpAQCAtODUsNyArODUsNyBAQAogCQkJCSAgTklM
X1AodGV4dCk/TlVMTDpTVFIyQ1NUUih0ZXh0KSwKIAkJCQkgIE5JTF9QKHR0ZXh0KT9OVUxMOlNU
UjJDU1RSKHR0ZXh0KSwKIAkJCQkgIE5JTF9QKHB0ZXh0KT9OVUxMOlNUUjJDU1RSKHB0ZXh0KSwK
LQkJCQkgIGdldF93aWRnZXQoaWNvbiksCisJCQkJICBOSUxfUChpY29uKT9OVUxMOmdldF93aWRn
ZXQoaWNvbiksCiAJCQkJICBleGVjX2NhbGxiYWNrLAogCQkJCSAgKGdwb2ludGVyKWZ1bmMsCiAJ
CQkJICBOVU0ySU5UKHBvcykpOwpAQCAtMjE1LDcgKzIxNSw3IEBACiB7CiAgICAgZ1Rvb2xiYXIg
PSByYl9kZWZpbmVfY2xhc3NfdW5kZXIobUd0aywgIlRvb2xiYXIiLCBnQ29udGFpbmVyKTsKIAot
ICAgIHJiX2RlZmluZV9jb25zdChnVG9vbGJhciwgIkNISUxEX1NQQ0FFIiwgSU5UMk5VTShHVEtf
VE9PTEJBUl9DSElMRF9TUEFDRSkpOworICAgIHJiX2RlZmluZV9jb25zdChnVG9vbGJhciwgIkNI
SUxEX1NQQUNFIiwgSU5UMk5VTShHVEtfVE9PTEJBUl9DSElMRF9TUEFDRSkpOwogICAgIHJiX2Rl
ZmluZV9jb25zdChnVG9vbGJhciwgIkNISUxEX0JVVFRPTiIsIElOVDJOVU0oR1RLX1RPT0xCQVJf
Q0hJTERfQlVUVE9OKSk7CiAgICAgcmJfZGVmaW5lX2NvbnN0KGdUb29sYmFyLCAiQ0hJTERfVE9H
R0xFQlVUVE9OIiwgSU5UMk5VTShHVEtfVE9PTEJBUl9DSElMRF9UT0dHTEVCVVRUT04pKTsKICAg
ICByYl9kZWZpbmVfY29uc3QoZ1Rvb2xiYXIsICJDSElMRF9SQURJT0JVVFRPTiIsIElOVDJOVU0o
R1RLX1RPT0xCQVJfQ0hJTERfUkFESU9CVVRUT04pKTsK

--Multipart_Sat__4_Aug_2001_01:15:15_+0900_0825d390--