From: MINAMI H. <mi...@ch...> - 2002-09-16 11:20:19
|
南です On Mon, 16 Sep 2002 19:53:41 +0900 Araki Ken <ara...@us...> wrote: > > とりあえず CVS HEAD で x_picture_set_icon_from_file() を削って、 > > x_window.c:x_window_set_icon()から x_picture_*.c の > > int x_picdep_load_icon( x_window_t *win, char *path, > > u_int32_t **cardinal, Pixmap *pixmap, Pixmap *mask) ; > > を直接呼ぶようにしました。 > なんとなく、x_picdep_load_icon の第一引数 win って必要なんでしょうか? > > x_picdep_load_file や x_picdep_load_background なんかは、ロードした背景 > 画像を貼り付ける window の情報が必要になるので、x_window_t を引数に渡し > てるわけですが、x_picdep_load_icon の場合は不要な気が... gdk-pixbuf では使ってないのですが、明示的に XCreatePixmap() する必要があると、 win->display が要りません? #というか gdk-pixbuf でもなしでよいはずがないので、 #xlib まわりはそのうち書きなおしが必要かも... > てか、その辺分かりやすいように、 > > x_picdep_load_file => x_picdep_load_file_for_background > x_picdep_load_background => x_picdep_get_transparent_background > x_picdep_load_icon => x_picdep_load_file > > あたりにするのはどうでしょう? > > あと、そもそも x_picdep_xxx って名前も x_imglib_xxx とかにしたいかも。 その方がよさそうですね。 |