Re: [patch] neovim patches

2017-04-18 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Di, 14 Mär 2017, Bram Moolenaar wrote: > > > On 14 March 2017 at 22:24, Christian Brabandt wrote: > > > > On Di, 14 Mär 2017, Christian Brabandt wrote: > > > > > Updated patch attached. > > > > I see that patch was already

Re: [patch] neovim patches

2017-04-17 Fir de Conversatie Christian Brabandt
Hi Bram! On Di, 14 Mär 2017, Bram Moolenaar wrote: > On 14 March 2017 at 22:24, Christian Brabandt wrote: > > On Di, 14 Mär 2017, Christian Brabandt wrote: > > > Updated patch attached. > > I see that patch was already included, also I did not receive an email

Re: [patch] neovim patches

2017-03-14 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: On 14 March 2017 at 22:24, Christian Brabandt wrote: > On Di, 14 Mär 2017, Christian Brabandt wrote: > > > Updated patch attached. > > I see that patch was already included, also I did not receive an email > yet. Please also add this part, which

Re: [patch] neovim patches

2017-03-14 Fir de Conversatie Christian Brabandt
On Di, 14 Mär 2017, Christian Brabandt wrote: > Updated patch attached. I see that patch was already included, also I did not receive an email yet. Please also add this part, which effectively reverts 7.4.700 and isn't needed anymore. (Sorry forget to squash this into the neovim patch.)

Re: [patch] neovim patches

2017-03-14 Fir de Conversatie Christian Brabandt
On Fr, 10 Mär 2017, Christian Brabandt wrote: > On Do, 09 Mär 2017, Christian Brabandt wrote: > > > Hi Bram! > > > > On Do, 09 Mär 2017, Bram Moolenaar wrote: > > > > > Found errors in Test_move_folds_around(): > > > function RunTheTest[21]..Test_move_folds_around line 26: Expected [-1, > > >

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Christian Brabandt
On Do, 09 Mär 2017, Christian Brabandt wrote: > Hi Bram! > > On Do, 09 Mär 2017, Bram Moolenaar wrote: > > > Found errors in Test_move_folds_around(): > > function RunTheTest[21]..Test_move_folds_around line 26: Expected [-1, -1, > > -1, -1, -1, -1, -1, 8, 8, 8, 8, 8, -1, 14, 14, 14, 14, 14]

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Christian Brabandt
Hi Bram! On Do, 09 Mär 2017, Bram Moolenaar wrote: > Found errors in Test_move_folds_around(): > function RunTheTest[21]..Test_move_folds_around line 26: Expected [-1, -1, > -1, -1, -1, -1, -1, 8, 8, 8, 8, 8, -1, 14, 14, 14, 14, 14] but got [-1, 2, 2, > 2, 2, 2, -1, 8, 8, 8, 8, 8, -1, 14, 14,

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Do, 09 Mär 2017, Bram Moolenaar wrote: > > > > > Christian Brabandt wrote: > > > > > > On Mo, 06 Mär 2017, Bram Moolenaar wrote: > > > > > > > > > > > > > > Christian Brabandt wrote: > > > > > > > > > > > And another one that fixes the :move closes folds and

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Christian Brabandt
Hi Bram! On Do, 09 Mär 2017, Bram Moolenaar wrote: > > Christian Brabandt wrote: > > > > On Mo, 06 Mär 2017, Bram Moolenaar wrote: > > > > > > > > > > > Christian Brabandt wrote: > > > > > > > > > And another one that fixes the :move closes folds and :move > > > > > invalidates > > > > >

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > > On Mo, 06 Mär 2017, Bram Moolenaar wrote: > > > > > > > > Christian Brabandt wrote: > > > > > > > And another one that fixes the :move closes folds and :move invalidates > > > > folds: > > > > https://github.com/neovim/neovim/pull/6221 > > > > > > It's a bit

Re: [patch] neovim patches

2017-03-09 Fir de Conversatie Christian Brabandt
On Mi, 08 Mär 2017, Christian Brabandt wrote: > On Mo, 06 Mär 2017, Bram Moolenaar wrote: > > > > > Christian Brabandt wrote: > > > > > And another one that fixes the :move closes folds and :move invalidates > > > folds: > > > https://github.com/neovim/neovim/pull/6221 > > > > It's a bit

Re: [patch] neovim patches

2017-03-08 Fir de Conversatie Bram Moolenaar
Suresh Govindachar wrote: > On 3/7/2017 4:10 PM, Nikolay Aleksandrovich Pavlov wrote: > > 2017-03-08 1:14 GMT+03:00 Bram Moolenaar : > >> Nikolay Pavlov wrote: > >>> 2017-03-05 16:56 GMT+03:00 Bram Moolenaar : > Justin M. Keyes wrote: > > On Sat,

Re: [patch] neovim patches

2017-03-08 Fir de Conversatie Christian Brabandt
On Mo, 06 Mär 2017, Bram Moolenaar wrote: > > Christian Brabandt wrote: > > > And another one that fixes the :move closes folds and :move invalidates > > folds: > > https://github.com/neovim/neovim/pull/6221 > > It's a bit big, but I suppose it's needed to make it work. > > Can someone turn

Re: [patch] neovim patches

2017-03-07 Fir de Conversatie 'Suresh Govindachar' via vim_dev
On 3/7/2017 4:10 PM, Nikolay Aleksandrovich Pavlov wrote: 2017-03-08 1:14 GMT+03:00 Bram Moolenaar : Nikolay Pavlov wrote: 2017-03-05 16:56 GMT+03:00 Bram Moolenaar : Justin M. Keyes wrote: On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt

Re: [patch] neovim patches

2017-03-07 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2017-03-08 1:14 GMT+03:00 Bram Moolenaar : > > Nikolay Pavlov wrote: > >> 2017-03-05 16:56 GMT+03:00 Bram Moolenaar : >> > >> > Justin M. Keyes wrote: >> > >> >> On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt >> >> wrote: >> >> >

Re: [patch] neovim patches

2017-03-07 Fir de Conversatie Bram Moolenaar
Nikolay Pavlov wrote: > 2017-03-05 16:56 GMT+03:00 Bram Moolenaar : > > > > Justin M. Keyes wrote: > > > >> On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt > >> wrote: > >> > On Sa, 04 Mär 2017, Bram Moolenaar wrote: > >> > > >> >> > >> >> Christian

Re: [patch] neovim patches

2017-03-06 Fir de Conversatie Nikolay Aleksandrovich Pavlov
2017-03-05 16:56 GMT+03:00 Bram Moolenaar : > > Justin M. Keyes wrote: > >> On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt >> wrote: >> > On Sa, 04 Mär 2017, Bram Moolenaar wrote: >> > >> >> >> >> Christian Brabandt wrote: >> >> >> >> > On Di, 28 Feb

Re: [patch] neovim patches

2017-03-06 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > And another one that fixes the :move closes folds and :move invalidates > folds: > https://github.com/neovim/neovim/pull/6221 It's a bit big, but I suppose it's needed to make it work. Can someone turn this into a Vim patch? And check that it works, removing the

Re: [patch] neovim patches

2017-03-06 Fir de Conversatie Christian Brabandt
And another one that fixes the :move closes folds and :move invalidates folds: https://github.com/neovim/neovim/pull/6221 Best, Christian -- Wenn uns die Mächtigen ihre Zuneigung bekunden, dann ist Wahlzeit. -- Lothar Schmidt -- -- You received this message from the "vim_dev"

Re: [patch] neovim patches

2017-03-05 Fir de Conversatie Bram Moolenaar
Justin M. Keyes wrote: > On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt > wrote: > > On Sa, 04 Mär 2017, Bram Moolenaar wrote: > > > >> > >> Christian Brabandt wrote: > >> > >> > On Di, 28 Feb 2017, Bram Moolenaar wrote: > >> > > But also testing fdm=marker requires

Re: [patch] neovim patches

2017-03-05 Fir de Conversatie Justin M. Keyes
On Sat, Mar 4, 2017 at 10:10 PM, Christian Brabandt wrote: > On Sa, 04 Mär 2017, Bram Moolenaar wrote: > >> >> Christian Brabandt wrote: >> >> > On Di, 28 Feb 2017, Bram Moolenaar wrote: >> > > But also testing fdm=marker requires more work, I'll leave that for >> > > now. >>

Re: [patch] neovim patches

2017-03-04 Fir de Conversatie Christian Brabandt
On Sa, 04 Mär 2017, Bram Moolenaar wrote: > > Christian Brabandt wrote: > > > On Di, 28 Feb 2017, Bram Moolenaar wrote: > > > But also testing fdm=marker requires more work, I'll leave that for > > > now. > > > > Are you sure? I think adding a :set fdm=marker and the rest should be > > the

Re: [patch] neovim patches

2017-03-04 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Di, 28 Feb 2017, Bram Moolenaar wrote: > > But also testing fdm=marker requires more work, I'll leave that for > > now. > > Are you sure? I think adding a :set fdm=marker and the rest should be > the same, foldmarkers will then be set automatically. Hmm, I

Re: [patch] neovim patches

2017-03-03 Fir de Conversatie Christian Brabandt
On Di, 28 Feb 2017, Bram Moolenaar wrote: > But also testing fdm=marker requires more work, I'll leave that for > now. Are you sure? I think adding a :set fdm=marker and the rest should be the same, foldmarkers will then be set automatically. BTW: There is another one:

Re: [patch] neovim patches

2017-02-28 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Another one: > https://github.com/neovim/neovim/pull/6194 > > I think the test case can be easily converted to a new style test. It > should be run once with fdm=manual and once with fdm=marker to see the > difference. Thanks. Yes, converting the test isn't

Re: [patch] neovim patches

2017-02-28 Fir de Conversatie Christian Brabandt
Another one: https://github.com/neovim/neovim/pull/6194 I think the test case can be easily converted to a new style test. It should be run once with fdm=manual and once with fdm=marker to see the difference. Best, Christian -- Sobald einer in einer Sache Meister geworden ist, sollte er in

Re: [patch] neovim patches

2017-01-02 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Here is another one, from neovim #5709 including a test Thanks! -- If Apple would build a car... ... it would be powered by the sun, be reliable, five times as fast and twice as easy to drive; but would only run on five percent of the roads. /// Bram Moolenaar

Re: [patch] neovim patches

2016-12-29 Fir de Conversatie Christian Brabandt
Here is another one, from neovim #5709 including a test Best, Christian -- In unserer Jugend schuften wir wie Sklaven, um etwas zu erreichen, wovon wir im Alter sorgenlos leben könnten; und wenn wir alt sind, sehen wir, daß es zu spät ist, so zu leben. -- Alexander Pope -- --

Re: [patch] neovim patches

2016-12-05 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > here is another patch from the neovim project: > > - https://github.com/neovim/neovim/pull/5717 > > A session does set winheight=1 winwidth=.. which fails to load, > if the user sets the winminheight/width option to larger values Thanks! -- >From "know your

Re: [patch] neovim patches

2016-12-05 Fir de Conversatie Christian Brabandt
Hi vim-dev! On Mo, 05 Dez 2016, Christian Brabandt wrote: > Bram, > here is another patch from the neovim project: > > - https://github.com/neovim/neovim/pull/5717 > > A session does set winheight=1 winwidth=.. which fails to load, > if the user sets the winminheight/width option to larger

Re: [patch] neovim patches

2016-12-05 Fir de Conversatie Christian Brabandt
Bram, here is another patch from the neovim project: - https://github.com/neovim/neovim/pull/5717 A session does set winheight=1 winwidth=.. which fails to load, if the user sets the winminheight/width option to larger values Best, Christian -- Nimm das Leben nicht so ernst. Du kommst sowieso

Re: [patch] neovim patches

2016-04-14 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Another one fixing an invalid memory access in utfc_ptr2char_len > > https://github.com/neovim/neovim/pull/4574 Perhaps. It doesn't have a test. And it depends on a C99 compiler. Can someone turn that into a patch for Vim? -- Facepalm statement #5: "Petrol

Re: [patch] neovim patches

2016-04-14 Fir de Conversatie Christian Brabandt
Another one fixing an invalid memory access in utfc_ptr2char_len https://github.com/neovim/neovim/pull/4574 Best, Christian -- Was verschenkt ein Mathematiker zu Weihnachten seiner Liebsten? Einen Polynomring in einer Intervallschachtelung verpackt. -- -- You received this message from the

Re: [patch] neovim patches

2016-03-06 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Another possible one: > https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4415.patch Yeah. The related code is a bit of a mess. The flag may only be used when running out of memory. -- Every engineer dreams about saving the universe and having sex

Re: [patch] neovim patches

2016-03-05 Fir de Conversatie Christian Brabandt
Another possible one: https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4415.patch Best, Christian -- Wie man sein Kind nicht nennen sollte: Lore Leyh -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are

Re: [patch] neovim patches

2016-02-23 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > another one: > https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4336.patch Thanks. Can be made shorter... -- Q: What's orange and sounds like a parrot? A: A carrot /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ ///

Re: [patch] neovim patches

2016-02-23 Fir de Conversatie Christian Brabandt
another one: https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/4336.patch Best, Christian -- Steht im Winter noch das Korn, ist es wohl vergessen wor'n. -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying

Re: [patch] neovim patches

2016-01-19 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Here is another one: > > https://github.com/neovim/neovim/pull/4047 Thanks! -- You have heard the saying that if you put a thousand monkeys in a room with a thousand typewriters and waited long enough, eventually you would have a room full of dead monkeys.

Re: [patch] neovim patches

2016-01-18 Fir de Conversatie Christian Brabandt
Here is another one: https://github.com/neovim/neovim/pull/4047 Best, Christian -- Wußten Sie schon... ... daß 500 Mark garnichts sind, besonders wenn man sie nicht hat? -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are

Re: [patch] neovim patches

2015-12-19 Fir de Conversatie Nikolay Aleksandrovich Pavlov
https://github.com/neovim/neovim/pull/3871 should be rather easy to port to Vim (except that I have no idea how to port tests). -- -- 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

Re: [patch] neovim patches

2015-11-14 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Here is a small doc-patch: > > https://github.com/neovim/neovim/pull/3660.patch Thanks. -- To be rich is not the end, but only a change of worries. /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ ///sponsor Vim, vote for

Re: [patch] neovim patches

2015-11-11 Fir de Conversatie Christian Brabandt
Here is a small doc-patch: https://github.com/neovim/neovim/pull/3660.patch Best, Christian -- -- 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

Re: [patch] neovim patches

2015-11-10 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > here is another one (possibly). > > https://github.com/neovim/neovim/pull/3614 > > Not sure, if this is correct, I ported it to vim, but it didn't seem to > have any effect. > > If anybody has a test case¹, that works, I'd like to hear so I can port > it to vim

Re: [patch] neovim patches

2015-11-10 Fir de Conversatie Bram Moolenaar
Hirohito Higashi wrote: > 2015-11-6(Fri) 19:55:44 UTC+9 h_east: > > Hi Christian! > > > > 2015-11-6(Fri) 17:28:03 UTC+9 Christian Brabandt: > > > Bram, > > > here is another one (possibly). > > > > > > https://github.com/neovim/neovim/pull/3614 > > > > > > Not sure, if this is correct, I

Re: [patch] neovim patches

2015-11-10 Fir de Conversatie Justin M. Keyes
On Tue, Nov 10, 2015 at 8:07 AM, Bram Moolenaar wrote: > > Christian Brabandt wrote: > >> here is another one (possibly). >> >> https://github.com/neovim/neovim/pull/3614 >> >> Not sure, if this is correct, I ported it to vim, but it didn't seem to >> have any effect. >> >> If

Re: [patch] neovim patches

2015-11-06 Fir de Conversatie h_east
Hi Christian! 2015-11-6(Fri) 17:28:03 UTC+9 Christian Brabandt: > Bram, > here is another one (possibly). > > https://github.com/neovim/neovim/pull/3614 > > Not sure, if this is correct, I ported it to vim, but it didn't seem to > have any effect. > > If anybody has a test case¹, that works,

Re: [patch] neovim patches

2015-11-06 Fir de Conversatie h_east
Hi Christian and Bram, 2015-11-6(Fri) 19:55:44 UTC+9 h_east: > Hi Christian! > > 2015-11-6(Fri) 17:28:03 UTC+9 Christian Brabandt: > > Bram, > > here is another one (possibly). > > > > https://github.com/neovim/neovim/pull/3614 > > > > Not sure, if this is correct, I ported it to vim, but it

Re: [patch] neovim patches

2015-09-09 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Issue: https://github.com/neovim/neovim/pull/3322 > Patch > https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/3322.patch > Problem is, Vim leaks memory, when 'wildignore' filters out all > matches, reproducible: vim -u NONE -N -c ':set

Re: [patch] neovim patches

2015-09-09 Fir de Conversatie Christian Brabandt
Bram, Issue: https://github.com/neovim/neovim/pull/3322 Patch https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/3322.patch Problem is, Vim leaks memory, when 'wildignore' filters out all matches, reproducible: vim -u NONE -N -c ':set wildignore=*|:!ls *' Best Christian --

Re: [patch] neovim patches

2015-05-05 Fir de Conversatie Christian Brabandt
Another small one: https://github.com/neovim/neovim/pull/2587#issuecomment-99221723 Best, Christian -- Der Geist des Alls ist gemeinschaftsfördernd. Wenigstens machte er die niedrigen Wesen wegen der höheren und paßte die höheren einandern an. -- Mark Aurel (Selbstbetrachtungen)

Re: [patch] neovim patches

2015-04-16 Fir de Conversatie Christian Brabandt
Hi vim-dev! On Do, 09 Apr 2015, Christian Brabandt wrote: Bram, here is another patch from the neovim project: - https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/2390.patch (makes :profile start ~/foobar expand the '~' and therefore prevent an error) Another mini

Re: [patch] neovim patches

2015-04-16 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: On Do, 09 Apr 2015, Christian Brabandt wrote: Bram, here is another patch from the neovim project: - https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/2390.patch (makes :profile start ~/foobar expand the '~' and therefore prevent an

Re: [patch] neovim patches

2015-04-11 Fir de Conversatie Bram Moolenaar
Christian wrote: here is another patch from the neovim project: - https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/2390.patch (makes :profile start ~/foobar expand the '~' and therefore prevent an error) I'll put it on the list. BTW: the last tag v7-4-692 is not

[patch] neovim patches

2015-04-09 Fir de Conversatie Christian Brabandt
Bram, here is another patch from the neovim project: - https://patch-diff.githubusercontent.com/raw/neovim/neovim/pull/2390.patch (makes :profile start ~/foobar expand the '~' and therefore prevent an error) BTW: the last tag v7-4-692 is not available on github yet. Best, Christian -- Wer