ruby-gnome2-devel-ja

[ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1

[Kouhei S. <ko...@co...>]

須藤です。

さっきの酒井さんのコミットの

> Index: rbglib_mainloop.c
> ===================================================================
> RCS file: /cvsroot/ruby-gnome2/ruby-gnome2/glib/src/rbglib_mainloop.c,v
> retrieving revision 1.6
> retrieving revision 1.7
> diff -u -d -r1.6 -r1.7
> --- rbglib_mainloop.c	29 Jun 2006 18:31:32 -0000	1.6
> +++ rbglib_mainloop.c	16 Jun 2007 02:46:28 -0000	1.7
> @@ -30,12 +30,17 @@
>  /*****************************************/
>  static VALUE rbglib_main_threads;
>  
> +/* FIXME */
> +#if 1
> +#define USE_POLL_FUNC 1
> +#else
>  #include <version.h>
>  #if RUBY_VERSION_CODE >= 154
>  # define USE_POLL_FUNC 1
>  #else
>  # undef USE_POLL_FUNC /* rb_thread_select() may cause busy wait */
>  #endif
> +#endif
>  
>  /* We can't use rbglib_poll() on native Win32.
>     Because GPollFD doesn't have file descriptor but HANDLE. */

なのですが、もしかしてコミットミスだったりします？

どんな影響があるのかよくわかっていないのですが、1.9対応のコー
ドにしては変だなぁと思っただけなのですが。。。

Re: [ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1

[Masahiro S. <sa...@to...>]

酒井です。

From: Kouhei Sutou <ko...@co...>
Date: Sat, 16 Jun 2007 11:57:27 +0900 (JST)

> 須藤です。
> 
> さっきの酒井さんのコミットの
> 
> > Index: rbglib_mainloop.c
> > ===================================================================
> > RCS file: /cvsroot/ruby-gnome2/ruby-gnome2/glib/src/rbglib_mainloop.c,v
> > retrieving revision 1.6
> > retrieving revision 1.7
> > diff -u -d -r1.6 -r1.7
> > --- rbglib_mainloop.c	29 Jun 2006 18:31:32 -0000	1.6
> > +++ rbglib_mainloop.c	16 Jun 2007 02:46:28 -0000	1.7
> > @@ -30,12 +30,17 @@
> >  /*****************************************/
> >  static VALUE rbglib_main_threads;
> >  
> > +/* FIXME */
> > +#if 1
> > +#define USE_POLL_FUNC 1
> > +#else
> >  #include <version.h>
> >  #if RUBY_VERSION_CODE >= 154
> >  # define USE_POLL_FUNC 1
> >  #else
> >  # undef USE_POLL_FUNC /* rb_thread_select() may cause busy wait */
> >  #endif
> > +#endif
> >  
> >  /* We can't use rbglib_poll() on native Win32.
> >     Because GPollFD doesn't have file descriptor but HANDLE. */
> 
> なのですが、もしかしてコミットミスだったりします？

すみません、コメントを書いておくべきでした。
これは意図的にこういうコードにしました。

> どんな影響があるのかよくわかっていないのですが、1.9対応のコー
> ドにしては変だなぁと思っただけなのですが。。。

現在の 1.9 では version.h がインストールされなくなっているようで、
バージョンを比較できないため、こうしました。

影響があるとしたら ruby 1.5.4 より前のバージョンですが、
そんな昔のバージョンでは現在のコードは元から動かないので、
問題はないはずです。

--
酒井 政裕 / Masahiro Sakai

Re: [ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1

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

むとうです。

On Sat, 16 Jun 2007 12:57:45 +0900 (JST)
Kouhei Sutou <ko...@co...> wrote:

> 須藤です。
> 
> > > どんな影響があるのかよくわかっていないのですが、1.9対応のコー
> > > ドにしては変だなぁと思っただけなのですが。。。
> > 
> > 現在の 1.9 では version.h がインストールされなくなっているようで、
> > バージョンを比較できないため、こうしました。

そういえばRubyKaigi2007でどなたかが言ってました。
version.hで見るのではなく、have_funcなどを使うように、
とのことですが、今回みたいなケースではやっぱりversion.h
なんかで判断したいですねー。
 
> > 影響があるとしたら ruby 1.5.4 より前のバージョンですが、
> > そんな昔のバージョンでは現在のコードは元から動かないので、
> > 問題はないはずです。
> 
> あぁ、なるほど。失礼しました。
> 
> であれば、もういっそUSE_POOL_FUNCごとごっそり消しちゃいませ
> んか？

#if 1 〜#endifの部分ですよね。1.5.4以前はもはやサポートする必要は
ないので削除しちゃって問題無しですね。さかいさんにお願いしちゃって
いいんでしょうか？

ちなみに、G_OS_WIN32の時はundefしてるので、USE_POLL_FUNC
自体はそのまま残すんですよね？

それでは。

Re: [ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1

[Masahiro S. () <sa...@to...>]

酒井です。

From: Masao Mutoh <mu...@hi...>
Date: Sat, 16 Jun 2007 13:34:22 +0900

> むとうです。
> 
> On Sat, 16 Jun 2007 12:57:45 +0900 (JST)
> Kouhei Sutou <ko...@co...> wrote:
> 
> > 須藤です。
> > 
> > > > どんな影響があるのかよくわかっていないのですが、1.9対応のコー
> > > > ドにしては変だなぁと思っただけなのですが。。。
> > > 
> > > 現在の 1.9 では version.h がインストールされなくなっているようで、
> > > バージョンを比較できないため、こうしました。
> 
> そういえばRubyKaigi2007でどなたかが言ってました。
> version.hで見るのではなく、have_funcなどを使うように、
> とのことですが、

がーん。

> 今回みたいなケースではやっぱりversion.h
> なんかで判断したいですねー。
> 
> > > 影響があるとしたら ruby 1.5.4 より前のバージョンですが、
> > > そんな昔のバージョンでは現在のコードは元から動かないので、
> > > 問題はないはずです。
> > 
> > あぁ、なるほど。失礼しました。
> > 
> > であれば、もういっそUSE_POOL_FUNCごとごっそり消しちゃいませ
> > んか？
> 
> #if 1 〜#endifの部分ですよね。1.5.4以前はもはやサポートする必要は
> ないので削除しちゃって問題無しですね。さかいさんにお願いしちゃって
> いいんでしょうか？

削除しました。

> ちなみに、G_OS_WIN32の時はundefしてるので、USE_POLL_FUNC
> 自体はそのまま残すんですよね？

そうです。

--
酒井 政裕 / Masahiro Sakai

Re: [ruby-gnome2-devel-ja] =?iso-2022-jp?b?cnVieS0xLjYueCA=?= =?iso-2022-jp?b?GyRCTVEkTiUzITwlSSROOm89fBsoQj8=?=

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

むとうです。

On Wed, 04 Jul 2007 02:45:33 +0900 (JST)
Masahiro Sakai (酒井政裕) <sa...@to...> wrote:

> 酒井です。
 
> USE_POLL_FUNCの話からは離れますが、
> ついでに 1.6.x と 1.7.x 用のコードも消してしまいたいところです。
> (例えばallocationフレームワーク以前のためのコードとか)
> 
> 1.6ももうサポートしていなかったですよね?

そうですね。削除しちゃいましょう。

Re: [ruby-gnome2-devel-ja] =?iso-2022-jp?b?cnVieS0xLjYueCA=?= =?iso-2022-jp?b?GyRCTVEkTiUzITwlSSROOm89fBsoQj8=?=

[Masahiro S. () <sa...@to...>]

From: Masao Mutoh <mu...@hi...>
Date: Wed, 4 Jul 2007 11:11:45 +0900

> むとうです。
> 
> On Wed, 04 Jul 2007 02:45:33 +0900 (JST)
> Masahiro Sakai (酒井政裕) <sa...@to...> wrote:
> 
> > 酒井です。
>  
> > USE_POLL_FUNCの話からは離れますが、
> > ついでに 1.6.x と 1.7.x 用のコードも消してしまいたいところです。
> > (例えばallocationフレームワーク以前のためのコードとか)
> > 
> > 1.6ももうサポートしていなかったですよね?
> 
> そうですね。削除しちゃいましょう。

削除しちゃいました。
ちょっとすっきり。

--
酒井 政裕 / Masahiro Sakai

[ruby-gnome2-devel-ja] =?iso-2022-jp?b?cnVieS0xLjYueCA=?= =?iso-2022-jp?b?GyRCTVEkTiUzITwlSSROOm89fBsoQj8=?=

[Masahiro S. () <sa...@to...>]

酒井です。

From: Masao Mutoh <mu...@hi...>
Subject: Re: [ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1
Date: Sat, 16 Jun 2007 13:34:22 +0900

> むとうです。

> > > 影響があるとしたら ruby 1.5.4 より前のバージョンですが、
> > > そんな昔のバージョンでは現在のコードは元から動かないので、
> > > 問題はないはずです。
> > 
> > あぁ、なるほど。失礼しました。
> > 
> > であれば、もういっそUSE_POOL_FUNCごとごっそり消しちゃいませ
> > んか？
> 
> #if 1 〜#endifの部分ですよね。1.5.4以前はもはやサポートする必要は
> ないので削除しちゃって問題無しですね。

USE_POLL_FUNCの話からは離れますが、
ついでに 1.6.x と 1.7.x 用のコードも消してしまいたいところです。
(例えばallocationフレームワーク以前のためのコードとか)

1.6ももうサポートしていなかったですよね?

--
酒井 政裕 / Masahiro Sakai

Re: [ruby-gnome2-devel-ja] #define USE_POLL_FUNC 1

[Kouhei S. <ko...@co...>]

須藤です。

> > どんな影響があるのかよくわかっていないのですが、1.9対応のコー
> > ドにしては変だなぁと思っただけなのですが。。。
> 
> 現在の 1.9 では version.h がインストールされなくなっているようで、
> バージョンを比較できないため、こうしました。
> 
> 影響があるとしたら ruby 1.5.4 より前のバージョンですが、
> そんな昔のバージョンでは現在のコードは元から動かないので、
> 問題はないはずです。

あぁ、なるほど。失礼しました。

であれば、もういっそUSE_POOL_FUNCごとごっそり消しちゃいませ
んか？