Avoid Vim's bug of :return in try-catch

[rhysd]

Vim script の :return {expr} の {expr} で例外が飛んでも :catch で補足されないようです．回避する修正を入れました．

• 報告： return の右項で例外が発生すると try/catch で補足されない。 issues#1135
• 修正： try/catch is ignored while error in return statement  vim/vim#2483

[ujihisa]

👍🍣

[rhysd]

一応もう1人ぐらいレビューしていただいたらマージします

[thinca]

ありがとうございます！正確には問題になるのは例外ではなく Vim のエラーです。
例えば以下のコードは問題ありません。

function! s:throw(msg) abort
  throw a:msg
endfunction

function! s:test() abort
  try
    return s:throw('foo')
  catch
    return 'ok'
  endtry
endfunction

echo s:test()
" => ok

呼び出した先でのエラーも大丈夫(内部構造理解してないですが、たぶん例外に変換されている?)。

function! s:throw() abort
  let _ = [] == 0
endfunction

function! s:test() abort
  try
    return s:throw()
  catch
    return 'ok'
  endtry
endfunction

echo s:test()
" => ok

[thinca]

という点を踏まえた上でも対策はしておいた方が安全かな…?

[rhysd]

すみません，どういうエラーが飛んでくるかまでは考慮せず問題が起こりそうな箇所はすべて回避するコードを入れている状態です．確認したほうが良いでしょうか

[rhysd]

ざっと全部見てみましたが，全箇所回避コードを入れなくて大丈夫そうですね．閉じます．@thinca さん，@ujihisa さん，レビューありがとうございます！

[tyru]

ret は Funcref だった場合も考えて Ret にすべきだと思います。

[tyru]

あと既存のグローバル関数名と被った時エラーにならないように l: 付けないとダメだった…
あとでやります。

[tyru]

あっ…すみません。コメント読んでませんでした…orz
確かに @thinca さんが言った再現条件だと回避コード入れなくてよさそうです。