Skip to content

Commit

Permalink
patch 9.0.2058: tests: avoid error when no swap files exist
Browse files Browse the repository at this point in the history
Problem:  tests: avoid error when no swap files exist
Solution: use unlet! so that no error message is reported
          in case the variable does not exists

When s:GetSwapFileList() does not find any swapfiles, it will return an
empty list []. This means, that the variable 'name' will not be
declared, cause the following unlet command to fail and causing a 1 sec
delay on running the tests.

So let's instead use the :unlet! command which simply skips reporting an
error when the variable given as parameter does not exists.

closes: #13396

Signed-off-by: Christian Brabandt <cb@256bit.org>
Co-authored-by: zeertzjq <zeertzjq@outlook.com>
  • Loading branch information
zeertzjq authored and chrisbra committed Oct 21, 2023
1 parent 3ec6c1f commit a36acb7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/testdir/runtest.vim
Expand Up @@ -215,7 +215,7 @@ endfunc
for name in s:GetSwapFileList()
call delete(name)
endfor
unlet name
unlet! name


" Invoked when a test takes too much time.
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -704,6 +704,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
2058,
/**/
2057,
/**/
Expand Down

0 comments on commit a36acb7

Please sign in to comment.