Re: [patch] updated breakindent patch

2014-06-30 Fir de Conversatie Christian Brabandt
Am 2014-06-30 21:46, schrieb Amadeus Demarzi: Christian, thanks for tackling a ton of little nitpicks I've had with Vim! One thing in general that would be awesome if fixed, especially in line with this patch, would be that when scrolling past lines that are wrapped, not having the document jum

Re: [patch] updated breakindent patch

2014-06-30 Fir de Conversatie Amadeus Demarzi
Christian, thanks for tackling a ton of little nitpicks I've had with Vim! One thing in general that would be awesome if fixed, especially in line with this patch, would be that when scrolling past lines that are wrapped, not having the document jump the number of lines the wrap take up. It make

Re: [patch] updated breakindent patch

2014-06-26 Fir de Conversatie Bram Moolenaar
Christian wrote: > On Mi, 25 Jun 2014, Bram Moolenaar wrote: > > > Bram wrote: > > > > > Christian wrote: > > > > > > > Finally a new patch, here we go: > > > > [...] > > > > > Thanks, much better now. > > > > > > Let me include it and make a few small improvements. I'm sure we will > > >

Re: [patch] updated breakindent patch

2014-06-26 Fir de Conversatie Bram Moolenaar
Christian - > On Mi, 25 Jun 2014, Bram Moolenaar wrote: > > > > Christian wrote: > > > > > > > Finally a new patch, here we go: > > > > [...] > > > > > Thanks, much better now. > > > > > > Let me include it and make a few small improvements. I'm sure we will > > > discover more problems onc

Re: [patch] updated breakindent patch

2014-06-25 Fir de Conversatie Christian Brabandt
On Mi, 25 Jun 2014, Bram Moolenaar wrote: > Bram wrote: > > > Christian wrote: > > > > > Finally a new patch, here we go: > > [...] > > > Thanks, much better now. > > > > Let me include it and make a few small improvements. I'm sure we will > > discover more problems once more people use th

Re: [patch] updated breakindent patch

2014-06-25 Fir de Conversatie Christian Brabandt
Hi Bram! On Mi, 25 Jun 2014, Bram Moolenaar wrote: > > Christian wrote: > > > > > Finally a new patch, here we go: > > [...] > > > Thanks, much better now. > > > > Let me include it and make a few small improvements. I'm sure we will > > discover more problems once more people use the featur

Re: [patch] updated breakindent patch

2014-06-25 Fir de Conversatie Bram Moolenaar
I wrote: > Christian wrote: > > > Finally a new patch, here we go: [...] > Thanks, much better now. > > Let me include it and make a few small improvements. I'm sure we will > discover more problems once more people use the feature, I hope you will > fix these problems then! I already found

Re: [patch] updated breakindent patch

2014-06-25 Fir de Conversatie Bram Moolenaar
Christian wrote: > Finally a new patch, here we go: > > On Do, 29 Mai 2014, Bram Moolenaar wrote: > > Christian wrote: > > > On Mi, 28 Mai 2014, Bram Moolenaar wrote: > > > > A few remarks from looking at the patch. > > > > > > > > There aren't any tests, while there are lots of changes to the

Re: [patch] updated breakindent patch

2014-06-23 Fir de Conversatie Christian Brabandt
Finally a new patch, here we go: On Do, 29 Mai 2014, Bram Moolenaar wrote: > Christian wrote: > > On Mi, 28 Mai 2014, Bram Moolenaar wrote: > > > A few remarks from looking at the patch. > > > > > > There aren't any tests, while there are lots of changes to the source > > > code. It would be rea

Re: [patch] updated breakindent patch

2014-05-29 Fir de Conversatie Bram Moolenaar
Christian wrote: > On Mi, 28 Mai 2014, Bram Moolenaar wrote: > > > Christian Brabandt wrote: > > > > > On Mo, 12 Mai 2014, Christian Brabandt wrote: > > > > > > > Thanks, I enhanced it further. You can adjust the behaviour now using > > > > :set breakindentopt > > > > > > For now, I put the

Re: [patch] updated breakindent patch

2014-05-28 Fir de Conversatie Michael Henry
On 05/28/2014 05:30 PM, Christian Brabandt wrote: > It will take some time, until I can work on those items. Probably not > within the next 2 weeks (I'll be on vacation with my kids, so Vim won't > be my priority the next couple of days ;)) Based on the time you must spend on your many patches, it

Re: [patch] updated breakindent patch

2014-05-28 Fir de Conversatie Christian Brabandt
Bram! On Mi, 28 Mai 2014, Bram Moolenaar wrote: > Christian Brabandt wrote: > > > On Mo, 12 Mai 2014, Christian Brabandt wrote: > > > > > Thanks, I enhanced it further. You can adjust the behaviour now using > > > :set breakindentopt > > > > For now, I put the patch here: > > > > https://raw

Re: [patch] updated breakindent patch

2014-05-28 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > On Mo, 12 Mai 2014, Christian Brabandt wrote: > > > Thanks, I enhanced it further. You can adjust the behaviour now using > > :set breakindentopt > > For now, I put the patch here: > > https://raw.githubusercontent.com/chrisbra/vim-mq-patches/master/breakindent_pat

Re: [patch] updated breakindent patch

2014-05-15 Fir de Conversatie Christian Brabandt
Hi On Mo, 12 Mai 2014, Christian Brabandt wrote: > Thanks, I enhanced it further. You can adjust the behaviour now using > :set breakindentopt For now, I put the patch here: https://raw.githubusercontent.com/chrisbra/vim-mq-patches/master/breakindent_patch I'll plan to keep it updated for a b

Re: [patch] updated breakindent patch

2014-05-12 Fir de Conversatie Christian Brabandt
On Fr, 09 Mai 2014, Ken Takata wrote: > Hi Christian, > > 2014/5/9 Fri 7:06:49 UTC+9 Christian Brabandt wrote: > > Hi Ken! > > > > On Mi, 07 Mai 2014, Ken Takata wrote: > > > > > 2014/5/8 Thu 6:50:22 UTC+9 Christian Brabandt wrote: > > > > Hi, > > > > here is an updated breakindent patch (brea

Re: [patch] updated breakindent patch

2014-05-09 Fir de Conversatie Ken Takata
Hi Christian, 2014/5/9 Fri 7:06:49 UTC+9 Christian Brabandt wrote: > Hi Ken! > > On Mi, 07 Mai 2014, Ken Takata wrote: > > > 2014/5/8 Thu 6:50:22 UTC+9 Christian Brabandt wrote: > > > Hi, > > > here is an updated breakindent patch (breakindent_patch.diff) for > > > current Vim. Changes include:

Re: [patch] updated breakindent patch

2014-05-09 Fir de Conversatie ZyX
> Here is an example: > The original patch does this: > , > | Lorem ipsum dolor sit amet, > | consetetur sadipscing elitr, > | sed > | Lorem ipsum dolor sit amet, > | >>consetetur sadipscing elitr, > | >>sed > ` > > While the second version makes it more like this: > , > |

Re: [patch] updated breakindent patch

2014-05-09 Fir de Conversatie Marius Gedminas
On Fri, May 09, 2014 at 12:06:49AM +0200, Christian Brabandt wrote: > 1 Question remains: > - Where to draw the 'sbr' setting? Does anybody have an opinion on that? I like your 2nd version better: > > > , > > > | Lorem ipsum dolor sit amet, > > > | consetetur sadipscing elitr, > > > | sed >

Re: [patch] updated breakindent patch

2014-05-08 Fir de Conversatie Christian Brabandt
Hi Ken! On Mi, 07 Mai 2014, Ken Takata wrote: > 2014/5/8 Thu 6:50:22 UTC+9 Christian Brabandt wrote: > > Hi, > > here is an updated breakindent patch (breakindent_patch.diff) for > > current Vim. Changes include: > > > > - displaywidth() function has an optional parameter, that takes the >

Re: [patch] updated breakindent patch

2014-05-08 Fir de Conversatie Bram Moolenaar
Christian Brabandt wrote: > Hi, > here is an updated breakindent patch (breakindent_patch.diff) for > current Vim. Changes include: > > - displaywidth() function has an optional parameter, that takes the > indent for that particular line into account > - correctly check for 'rela

Re: [patch] updated breakindent patch

2014-05-07 Fir de Conversatie Ken Takata
Hi Christian, 2014/5/8 Thu 6:50:22 UTC+9 Christian Brabandt wrote: > Hi, > here is an updated breakindent patch (breakindent_patch.diff) for > current Vim. Changes include: > > - displaywidth() function has an optional parameter, that takes the > indent for that particular line into a

[patch] updated breakindent patch

2014-05-07 Fir de Conversatie Christian Brabandt
Hi, here is an updated breakindent patch (breakindent_patch.diff) for current Vim. Changes include: - displaywidth() function has an optional parameter, that takes the indent for that particular line into account - correctly check for 'relativenumber' option (and not only number