Dominique wrote:
> Yegappan Lakshmanan <[email protected]> wrote: > > > Hi Bram, > > > > On Thu, Nov 12, 2020 at 9:08 AM Bram Moolenaar <[email protected]> wrote: > >> > >> > >> I wrote: > >> > >> > Patch 8.2.1978 > >> > Problem: Making a mapping work in all modes is complicated. > >> > Solution: Add the <Cmd> special key. (Yegappan Lakshmanan, closes > >> > #7282, > >> > closes 4784, based on patch by Bjorn Linse) > >> > Files: runtime/doc/autocmd.txt, runtime/doc/eval.txt, > >> > runtime/doc/map.txt, src/edit.c, src/errors.h, > >> > src/ex_docmd.c, > >> > src/ex_getln.c, src/getchar.c, src/insexpand.c, src/keymap.h, > >> > src/map.c, src/misc2.c, src/normal.c, src/ops.c, > >> > src/proto/getchar.pro, src/screen.c, src/terminal.c, > >> > src/testdir/test_mapping.vim > >> > >> Somehow the Appveyor test hangs in test_swap since this patch. I have > >> no idea why. Can someone reproduce this on MS-Windows? > >> > >> > > > > I saw this problem and tried to reproduce this on MS-Windows and couldn't > > reproduce it so far. > > How about making test verbose in CI, at least temporarily > to help fix this issue? like this: > > diff --git a/src/testdir/Makefile b/src/testdir/Makefile > index 2fb329aa1..6431f6a6b 100644 > --- a/src/testdir/Makefile > +++ b/src/testdir/Makefile > @@ -12,7 +12,7 @@ SCRIPTSOURCE = ../../runtime > # Comment out this line to see the verbose output of tests. > # > # Catches SwapExists to avoid hanging at the ATTENTION prompt. > -REDIR_TEST_TO_NULL = --cmd 'au SwapExists * let v:swapchoice = "e"' > > /dev/null > +#REDIR_TEST_TO_NULL = --cmd 'au SwapExists * let v:swapchoice = "e"' > > /dev/null > > # Uncomment this line to use valgrind for memory leaks and extra warnings. > # The output goes into a file "valgrind.testN" It looks like the test passes again. Both for Yegappan's pull request and for patch 8.2.1980. I just found a nasty problem when sourcing the same script again: the hashtable for local variables was not initialized. That caused a hang in test_vim9_expr. It would be very unexpected if the test_swap problem was related to this though. -- hundred-and-one symptoms of being an internet addict: 242. You turn down a better-paying job because it doesn't come with a free e-mail account. /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org /// -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/202011121927.0ACJRrra113093%40masaka.moolenaar.net.
