bufhiddenを設定するとBufUnloadが2回実行される

[norio13]

質問・報告の内容

bufhiddenオプションにunload、deleteのどちらかが設定されたバッファに対して、
:bunload、:bdelete、:bwipeoutを実行するとBufUnloadが2回発火されました。

bufhiddenの値がhideとwipeまたは値なしの場合、上記3つのコマンドを実行してもBufUnloadは1回のみ発火します。

再現方法

Vim起動

vim -Nu NONE test.vim -c "so %"

test.vimの内容

new

" setlocal bufhidden=unload
setlocal bufhidden=delete

augroup mygroup
    autocmd!
    autocmd BufUnload <buffer> echo 'BufUnload'
augroup END

bunload
" bdelete
" bwipeout

Vimのバージョン

7.4.1830

OSの種類

• Linux

OSのディストリ

Debian

OSのバージョン

8.4

[h-east]

これで様子を見てみて下さい。
make testはALL DONEなのは確認しました。

diff --git a/src/buffer.c b/src/buffer.c
index 9bc24bc..170e214 100644
--- a/src/buffer.c
+++ b/src/buffer.c
@@ -574,9 +574,12 @@ buf_freeall(buf_T *buf, int flags)
     int        is_curbuf = (buf == curbuf);

     buf->b_closing = TRUE;
-    apply_autocmds(EVENT_BUFUNLOAD, buf->b_fname, buf->b_fname, FALSE, buf);
-    if (!buf_valid(buf))       /* autocommands may delete the buffer */
-   return;
+    if (buf->b_ml.ml_mfp != NULL)
+    {
+   apply_autocmds(EVENT_BUFUNLOAD, buf->b_fname, buf->b_fname, FALSE, buf);
+   if (!buf_valid(buf))        /* autocommands may delete the buffer */
+       return;
+    }
     if ((flags & BFA_DEL) && buf->b_p_bl)
     {
    apply_autocmds(EVENT_BUFDELETE, buf->b_fname, buf->b_fname, FALSE, buf);

[mattn]

これ系多いすね。まだありそう。。。

[norio13]

@h-east patch適用後、BufUnloadが2回発生しなくなりました。
迅速な対応に感謝致します。
ありがとうございます。

[h-east]

testを追加してvim_devに投げました。
https://groups.google.com/d/msg/vim_dev/xqWcI1YY9Qc/UTD83XpOCAAJ

[h-east]

patch 7.4.1837 👍
vim/vim@c67e892

@norio13 issue登録ありがとうございました。