Kazunobu Kuriyama wrote: > Unfortunately, I got the same experience that Manuel Ortega went through. > > Yesterday, I ran "make test" several times after pulling the patch into my > repo, and didn't see any failure with it. > > But today, I got it (FWIW, after pulling other new patches into the repo > and building vim with them). > > The error message I got was the same as Manuel's: > > >From test_channel.vim: > Found errors in Test_open_delay(): > function Test_open_delay[3]..<SNR>3_run_server line 46: 'Caught exception: > Vim(call):E631: ch_evalexpr(): write failed' > TEST FAILURE > > As Bram suggested, I edited test_channel.vim to have the log file. But > then the error was gone as Manuel described.
Darn. Must be a timing problem. > The bug is really nasty. It usually behaves as if it were normal but > suddenly hits unlucky guys. If someone tries to inspect it, it goes out of > sight. > > I feel the patch should be revised or reverted. The main difference after the patch is that "len" isn't checked to be zero. Can you put that back and see if it helps? -- SECOND SOLDIER: It could be carried by an African swallow! FIRST SOLDIER: Oh yes! An African swallow maybe ... but not a European swallow. that's my point. "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD /// Bram Moolenaar -- b...@moolenaar.net -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.