vim.exe で channel が有効にならない #809

mattn · 2016-01-29T05:28:14Z

非サポート扱いの様だったので直しました。

既知の問題点として(gvimでもあり得る)、json を使う場合、1回の受信で2回分受信してしまった時に json_decode がエラーになる(当然)ので、処理性能の遅い vim.exe が急にソケット受信するとエラーになる。

どなたか検証お願いします。

tyru · 2016-01-29T05:36:20Z

👍

k-takata · 2016-01-29T06:02:03Z

試していませんが、HAVE_SELECT の定義の仕方が気になりました。
select() があるかどうかの定義なので、GUI/CUI で定義を変えるのは不自然だと思います。

mattn · 2016-01-29T06:42:40Z

FEAT_GUI_W32 の場合は HAVE_SELECT じゃないパスなんですよね。。。

k-takata · 2016-01-29T06:47:58Z

長くなってしまいますが、 defined(HAVE_SELECT) && !defined(FEAT_GUI_W32) でいいんじゃないでしょうか。こっちの方が分かりやすいと思います。

mattn · 2016-01-29T06:58:39Z

長くなってしまいますが、 defined(HAVE_SELECT) && !defined(FEAT_GUI_W32) でいいんじゃないでしょうか。こっちの方が分かりやすいと思います

どの部分でしょうか？

mattn · 2016-01-29T07:11:18Z

あ、わかりました。

mattn · 2016-01-29T07:12:54Z

k-takata · 2016-01-29T07:15:28Z

あれ、vim.h の HAVE_SELECT のdefine部分はそのままですか？

mattn · 2016-01-29T07:24:33Z

Windows だと HAVE_SELECT は自分で定義する必要があるのです。
Windows GUI は HAVE_SELECT に依存していないので、Windows Console の場合は自分で定義しています。

k-takata · 2016-01-29T08:36:51Z

GUIでも select() を使いたくなったときに、はまりそうです。CUI/GUIで分けないほうがよいと思います。
channel_wait() の動作が HAVE_SELECT で変わりそうですが、大丈夫ですか？

mattn · 2016-01-29T09:36:06Z

gvim.exe と vim.exe で channel の実装方式が違うのです。
コード読んでもらえると分かるのですが、gvim はウィンドウハンドルを使って select する方式を取っています。
channel の目的では 混ぜる事はないと思います。

mattn · 2016-01-29T09:38:16Z

extern かな。直しておきます。

mattn · 2016-01-29T10:05:23Z

GUI 版は channel_wait に来た時点で select は完了しているので(そもそもトリガがウィンドウハンドルによるselect)、敢えて HAVE_SELECT を実行してないんだと思います。

k-takata · 2016-01-29T20:21:21Z

mattn · 2016-02-02T01:37:11Z

↑に書いたJSONを2回受け取るとパースが出る件はBramが直してくれた。

k-takata added the related/windows label Jan 29, 2016

k-takata closed this as completed Jan 29, 2016

k-takata added status/sent patch xlose/fixed labels Jan 29, 2016

Provide feedback