diff options
| author | Justin M. Keyes <justinkz@gmail.com> | 2019-08-11 13:34:44 +0200 |
|---|---|---|
| committer | Justin M. Keyes <justinkz@gmail.com> | 2019-08-12 02:17:49 +0200 |
| commit | 71378a40308cd987626ebccb5882897d41c86c7a (patch) | |
| tree | 0341f2fb5ac5e0813986e740ea512e99a6cd80f0 /src/nvim/testdir | |
| parent | feee862064554a789891e3450762464a6ec2535d (diff) | |
| download | rneovim-71378a40308cd987626ebccb5882897d41c86c7a.tar.gz rneovim-71378a40308cd987626ebccb5882897d41c86c7a.tar.bz2 rneovim-71378a40308cd987626ebccb5882897d41c86c7a.zip | |
free_buffer: rework b:changedtick handling #9163
- Re-enable Test_BufLeave_Wipe. 66f5e5c7d7ce
This is my (probably-wrong) application of Zyx's suggestion in #9163:
> free_buffer_stuff() already removes changedtick. It is better to
> make `free_options` a flag variable and avoid calling
> buf_init_changedtick() based on some flag there: current workflow
> looks weird as it first removes `b:changedtick`, then re-adds it
> by calling buf_init_changedtick(), then remove again.
> Also based on my understanding it looks logical to not remove
> `b:changedtick`, but to *replace* it with something allocated if
> needed based on examining reference count before calling
> `unref_var_dict`. Because now you have key disappearing from
> dictionary for no good reason.
Patch-By: Nikolai Aleksandrovich Pavlov <kp-pav@yandex.ru>
Diffstat (limited to 'src/nvim/testdir')
| -rw-r--r-- | src/nvim/testdir/test_autocmd.vim | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/nvim/testdir/test_autocmd.vim b/src/nvim/testdir/test_autocmd.vim index b686d0292f..4c9df0cb16 100644 --- a/src/nvim/testdir/test_autocmd.vim +++ b/src/nvim/testdir/test_autocmd.vim @@ -811,7 +811,6 @@ endfunc " Test for autocommand that deletes the current buffer on BufLeave event. " Also test deleting the last buffer, should give a new, empty buffer. func Test_BufLeave_Wipe() - throw 'skipped: TODO: ' %bwipe! let content = ['start of test file Xxx', \ 'this is a test', |