Re: [ruby-gnome2-devel-ja] vteの実装をしてみました

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

須藤です。

In <CAM...@ma...>
  "Re: [ruby-gnome2-devel-ja] vteの実装をしてみました" on Sun, 11 Sep 2011 01:29:13 +0900,
  Masaaki Aoyagi <mas...@gm...> wrote:

>> rb_grn_*()は関数のprefixをrb_grnではなく、Ruby-GNOME2のやり
>> 方にあわせて変えてしまってOKです！
> 
> Hash引数は他にも使うところが出てきてglibに移動することになると踏んでるので、
> その時に場所や名前を考えればいいかなということで、そのままの名前にしておきました。

なるほど！
それでよいと思います！

>> vte_terminal_fork_command_full()も__vte_pty_spawn()もどちら
>> もchild_setupにVtePtyを渡してくれないのでだいぶ厳しいですね。
>>
>> 思いついたのは、こんなのですが、大変なので、とりあえず、
>> child_setupは未サポートがいいんじゃないかと思います。
...
> 
> やっぱり、ややこしい事をしないといけないのですね。
> 未サポートにしました。

はい、それが現実的かと思います。

>> あと、（うろ覚えですが）いくつか気になった所があるのですが、
>> これはコミット後に直すのでもいいかなぁと思っています。
>>
>>  * NIL_P(xxx) ? NULL : RVAL2CSTR(xxx)の代わりに
>>    RVAL2CSTR_ACCEPT_NIL(xxx)というのが使えた気がする。
>>    （うろ覚え）
>>  * NIL_P(xxx) ? NULL : RVAL2GOBJ(xxx)は
>>    RVAL2GOBJ(xxx)だけでいけた気がする。
>>    （うろ覚え）
> 
> この手のパターンは色々な所で使っていそうなので、どこかのタイミングで
> 一気に作業した方がいいかなと思います。

わかりました！

>>> 今回対象となるメソッドは、search_set_gregex と search_set_wrap_around ですが、
>>> 代入メソッドを定義した方が良いでしょうか？
>>
>> あった方がRubyっぽく書けるような気がするのでお願いします。
> 
> エイリアスを定義するようにしました。

ありがとうございます！

>> search_set_gregex()とsearch_get_gregex()について少し考えてみ
>> たのですが、現状はGLib::Regexpがなく、今後も追加するかどうか
>> はまだ検討する余地があると思う(*)ので、今のところは以下のよ
>> うにするのはいかがでしょうか？
>>
>>  * search_set_gregex: 引数は文字列を受け取り、内部でGRegexp
>>    を生成して使う。
>>  * search_get_gregex: APIを提供しない。あるいは
>>    g_regex_get_pattern()の値を返す。
>>
>> (*) RubyのRegexpとGLib::Regexがあると混乱しそうとか、Rubyの
>>    Regexpの方が断然使いやすいのでGLib::Regexがある意味があ
>>    るのかとか。
> 
> match_add_gregexもあるので、とりあえずGRegex関連は実装を見合わせました。

はい。必要になったらまた検討することにしましょう。

>>> fork_command_full を外に出さないようにしようと考えています。
>>
>> 私もそれがいいと思います！
> 
> メソッド定義を削除しました。

ありがとうございます！

> 以上の内容でコミットしましたので、ご確認ください。

確認しました！
よいと思います！