Re: "ch_read()" couldn't tell the difference between "not enough message" and "a empty line"

2016-11-25 Fir de Conversatie skywind3000
What about provide an option "nl" in ch_read to make it better ? :call ch_read(mych, {"timeout": 0, "nl":1}) returns "abc\n" for a line "abc" returns "\n" for an empty line returns "" for not enough data This will not break the plugins, and will make ch_read better. It could be used in a timer

Re: [vim/vim] Empty first tab lost when using sessions (#1282)

2016-11-25 Fir de Conversatie h_east
Hi Jim and list, 2016-11-26(Sat) 5:40:52 UTC+9 vim-dev ML: > > I can reproduce it. > > > And I wrote a patch for this issue. (Attached) > > > Could you please check this patch? > > > When issue has been resolved, I'll add a test later. > > > > This patch fixes it for me, with 8.0.52. I'm

Re: [vim/vim] Empty first tab lost when using sessions (#1282)

2016-11-25 Fir de Conversatie Jim Stewart
> I can reproduce it. > And I wrote a patch for this issue. (Attached) > Could you please check this patch? > When issue has been resolved, I'll add a test later. This patch fixes it for me, with 8.0.52. I'm not familiar with the Vim source, but if there's anything I can do to help, please let

Re: Patch 8.0.0101

2016-11-25 Fir de Conversatie Bram Moolenaar
Daisuke Suzuki wrote: > > Ah, sorry. The flag name is a bit confusing, as file name normally > > means including path separators, but here it really means only the file > > name. I think we should add another one for path names. > > > > Should also have a test for this... > > Hi Bram, > > I

Patch 8.0.0102

2016-11-25 Fir de Conversatie Bram Moolenaar
Patch 8.0.0102 (after 8.0.0101) Problem:Cannot set 'dictionary' to a path. Solution: Allow for slash and backslash. Add a test (partly by Daisuke Suzuki, closes #1279, closes #1284) Files: src/option.c, src/testdir/test_options.vim *** ../vim-8.0.0101/src/option.c

Re: [vim/vim] Empty first tab lost when using sessions (#1282)

2016-11-25 Fir de Conversatie h_east
Hi Jim! 2016-11-26(Sat) 0:41:46 UTC+9 Jim Stewart: > I'm creating a session with mksession, and when I restore the session, if the > first tab was empty ([No Name]), it's gone in the restored session. Other > empty tabs are restored fine; it seems to only be the first tab that's lost. > If

Re: Patch 8.0.0101

2016-11-25 Fir de Conversatie Daisuke Suzuki
> Ah, sorry. The flag name is a bit confusing, as file name normally > means including path separators, but here it really means only the file > name. I think we should add another one for path names. > > Should also have a test for this... Hi Bram, I created a minimal test for 'dictionary'.

Re: Patch 8.0.0101

2016-11-25 Fir de Conversatie Bram Moolenaar
Mike Williams wrote: > On 24/11/2016 20:46, Bram Moolenaar wrote: > > > > Patch 8.0.0101 > > Problem:Some options are not strictly checked. > > Solution: Add flags for strickter checks. > > Files: src/option.c > > > > > > *** ../vim-8.0.0100/src/option.c2016-11-24

Re: "ch_read()" couldn't tell the difference between "not enough message" and "a empty line"

2016-11-25 Fir de Conversatie skywind3000
Thanks for the patch, I wrote a test : (benchjob.vim for starting job, benchjob.py for printing 8 lines) --- benchjob.vim: function! MyCallback(job, text) " make cpu a little busy for i in range(100) for j in range(10)

Re: Patch 8.0.0101

2016-11-25 Fir de Conversatie Mike Williams
Hi On 24/11/2016 20:46, Bram Moolenaar wrote: Patch 8.0.0101 Problem:Some options are not strictly checked. Solution: Add flags for strickter checks. Files: src/option.c *** ../vim-8.0.0100/src/option.c2016-11-24 18:30:55.715444785 +0100 --- src/option.c2016-11-24