2016-04-28 4:47 GMT+09:00 Bram Moolenaar <b...@moolenaar.net>:

>
> Kazunobu Kuriyama wrote:
>
> > 2016-04-28 0:42 GMT+09:00 Bram Moolenaar <b...@moolenaar.net>:
> >
> > >
> > > Kazunobu Kuriyama wrote:
> > >
> > > > 2016-04-27 17:51 GMT+09:00 Bram Moolenaar <b...@moolenaar.net>:
> > > >
> > > > >
> > > > > Christian Brabandt wrote:
> > > > >
> > > > > > Am 2016-04-26 19:01, schrieb Bram Moolenaar:
> > > > > > > Patch 7.4.1789
> > > > > >
> > > > > > I think this patch broke the vim-win32-installer build
> > > > > >
> > > > > >   From test_channel.vim:
> > > > > > Found errors in Test_out_close_cb():
> > > > > > function RunTheTest[9]..Test_out_close_cb line 23: Expected 1 but
> > > got 2
> > > > > > function RunTheTest[9]..Test_out_close_cb line 24: Expected 2 but
> > > got 3
> > > > > > TEST FAILURE
> > > > > > NMAKE : fatal error U1077: 'if' : return code '0x1'
> > > > > > Stop.
> > > > > > Command exited with code 1
> > > > > >
> > > > > >
> > > > >
> > >
> https://ci.appveyor.com/project/chrisbra/vim-win32-installer/build/94/job/ah9t0xb7hkh14ous
> > > > >
> > > > > I noticed the travis build also failed with this error, but then
> passed
> > > > > later.  For me it always passes, thus it's hard to find out why it
> > > would
> > > > > sometimes fail.  It looks like the callback is invoked twice, but I
> > > > > don't know why/when that happens.  Any idea?
> > > > >
> > > >
> > > > On OS X, the same error happens occasionally.
> > > >
> > > > Since it didn't always happen, I had put it on probation and
> refrained
> > > from
> > > > accusing it of the failure. :)
> > > >
> > > > Having done 'make test' five or six times with ch_logfile of
> > > > test_channel.vim enabled, I finally caught it.
> > > >
> > > > Attached are the waring message of the failure (warning.txt) and
> > > > testdir/channellog when it happened.
> > > >
> > > > Hopefully, this could be helpful for you.
> > >
> > > Unfortunately the channellog is truncated before the interesting part.
> > > Perhaps you can start the log in the Test_out_close_cb function.
> > >
> >
> > I added call
> >
> >     ch_log('Test_out_close_cb()')
> >
> > just after test_channel.vim:1055, and the attached file is the result.
>
> That line was already there.  do you mean you added ch_logfile()?
>

Exactly.  More precisely,

diff --git a/src/testdir/test_channel.vim b/src/testdir/test_channel.vim
index 6eb3c1f..1e6db68 100644
--- a/src/testdir/test_channel.vim
+++ b/src/testdir/test_channel.vim
@@ -1052,6 +1052,7 @@ func Test_out_close_cb()
   if !has('job')
     return
   endif
+  call ch_logfile('channellog', 'w')
   call ch_log('Test_out_close_cb()')

   let s:counter = 1


> > Does this contain the part you need?
>
> This is what I see:
>
>         ==== start log session ====
>           0.000015 : Test_out_close_cb()
>           0.000114 : Starting job: python  test_channel_pipe.py  quit  now
>           0.000148 on 26: Created channel
>           0.012371 ERR on 25: channel_select_check(): Cannot read from
> channel
>           0.012463 on 25: Closing channel
>
> 25 is from the previous test, we can ignore that.
>
>           0.012479 : looking for messages on channels
>           0.038454 RECV on 26: 'quit
>         '
>           0.038475 on 26: Invoking channel callback OutHandler
>
> First call to OutHandler
>
>           0.049656 ERR on 26: channel_select_check(): Cannot read from
> channel
>           0.049742 PUT on 26: 'DETACH
>         '
>           0.049750 on 26: Closing channel
>           0.049805 on 26: Invoking channel callback OutHandler
>
> Second call to OutHandler
>
>           0.049953 on 26: Invoking close callback CloseHandler
>
> Hmm, it does appear to call OutHandler twice.
> In my case the first call isn't there.
> I suppose what happens is that the DETACH is received separately.
> That is actually what should happen, so why is that message dropped in
> my case?  Requires some more debugging...
>

If you need more experiments, or something I can do for you, let me know.

>
> --
> ARTHUR: Charge!
>    [They all charge with swords drawn towards the RABBIT.  A tremendous
> twenty
>    second fight with Peckinpahish shots and borrowing heavily also on the
>    Kung Fu and karate-type films ensues, in which some four KNIGHTS are
>    comprehensively killed.]
> ARTHUR: Run away!  Run away!
>                  "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.

Raspunde prin e-mail lui