Re: [ruby-gnome2-devel-ja] gdk_atom_intern_static_string (Re: [ruby-gnome2-cvs] CVS: ruby-gnome2/gtk/src rbgdkatom.c, 1.12, 1.13 rbgdkcairo.c, 1.2, 1.3 rbgdkdisplay.c, 1.19, 1.20 rbgdkpixmap.c, 1.25, 1.26 rbgdkscreen.c, 1.17, 1.18 rbgdkwindow.c, 1.60, 1.61)

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

むとうです。

On Sun, 15 Jul 2007 17:40:56 +0900 (JST)
Masahiro Sakai (酒井政裕) <sa...@to...> wrote:

> 酒井です。
> 
> From: Masao Mutoh <mu...@us...>
> Subject: [ruby-gnome2-cvs] CVS: ruby-gnome2/gtk/src rbgdkatom.c, 1.12, 1.13 rbgdkcairo.c, 1.2, 1.3 rbgdkdisplay.c, 1.19, 1.20 rbgdkpixmap.c, 1.25, 1.26 rbgdkscreen.c, 1.17, 1.18 rbgdkwindow.c, 1.60, 1.61
> Date: Sat, 25 Nov 2006 09:50:44 -0800
> 
> > Log Message:
> 
> > 	* src/rbgdkatom.c: Added Gdk::Atom.intern_static_string for GTK+-2.10. 
> 
> > Index: rbgdkatom.c
> > ===================================================================
> > RCS file: /cvsroot/ruby-gnome2/ruby-gnome2/gtk/src/rbgdkatom.c,v
> > retrieving revision 1.12
> > retrieving revision 1.13
> > diff -u -d -r1.12 -r1.13
> > --- rbgdkatom.c	30 Jul 2005 11:22:15 -0000	1.12
> > +++ rbgdkatom.c	25 Nov 2006 17:50:41 -0000	1.13
> > @@ -61,6 +61,16 @@
> >                        GDK_TYPE_ATOM);
> >  }
> >  
> > +#if GTK_CHECK_VERSION(2,10,0)
> > +static VALUE
> > +gdkatom_s_intern_static_string(self, name)
> > +    VALUE self, name;
> > +{
> > +    return BOXED2RVAL(gdk_atom_intern_static_string(RVAL2CSTR(name)), 
> > +                      GDK_TYPE_ATOM);
> > +}
> > +#endif
> > +
> 
> gdk_atom_intern_static_string() は引数となる文字列が存在し続けることを
> 前提として、コピーを行わないというメソッドなので、こういう使い方はまず
> いです。
> http://developer.gnome.org/doc/API/2.2/gdk/gdk-Properties-and-Atoms.html#gdk-atom-intern-static-string
> 
> Gdk::Atom.intern_static_string は不要でしょう。

そうですね。
まだ使っている人も少ないでしょうから削除しておいていただけますか。

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