thinca / vim-ref

Integrated reference viewer.
204 stars 39 forks source link

Fix ref#system for Windows #62

Closed erw7 closed 8 years ago

erw7 commented 8 years ago

Windows で sxq, sxe が設定されている場合、call_shell の処理でコマンドが二重にクォート・エスケープされてしまい ref#system が正常に動作しないようだったので修正しました。 nvim については libuv_process_spawn で UV_PROCESS_WINDOWS_VERBATIM_ARGUMENT が指定されているため外側の " は必要なようです。処理を追いきれなかったのでよくわからないのですが、C:\^a b というディレクトリを作成したうえで中に test.txt を作成し、

sysmtem('""ls" "C:\^a b""')

してみたところ、test.txt が表示されましたので &<>()@^| の ^ でのエスケープ処理はされているようです。

よろしければ取り込んで頂けますか。

thinca commented 8 years ago

報告ありがとうございます。返事遅れてすいません。 おっしゃる通り 'shellxquote' のデフォルト値が変わったりした関係でうまく動かなくなっていたので、 b45d10f6d0c82e3afd17774626e952fde03274a1 で修正しました。 頂いた変更ですが、申し訳ないですが ref.vim は Neovim をサポートする予定はありません。これは私自身 Neovim を使っておらず、メンテナンスしていくのが難しいためです。この変更を取り込んでしまうと事実上 Neovim をサポートすることになってしまうので、こちらで修正させて頂きました。 また、Neovim で動かなかったとしてもサポート予定はありませんのでご了承ください。Neovim という亜種版を使うリスクとしてご理解頂ければと思います。

erw7 commented 8 years ago

ご対応ありがとうございました。 Neovim の件については了解しました。