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 account
>     - correctly check for 'relativenumber' option (and not only number 
>       option)
>     - in diff mode, highlight the indent accordingly. (Note: In diff 
>       mode, there seems to be a bug in Vim, that the filler lines are 
>       not correctly calculated, when wrap is on. This needs to be fixed 
>       independently of this patch, not that wrapping makes much sense in 
>       diff mode anyhow)
>     - style adjusted to match more the Vim style
> 
> Since I personally prefer, that the 'sbr' setting will still be drawn at 
> the start of the screen line, rather then after the indent, here is a 
> second patch (sbr_linestart_breakindent.diff), that makes this the 
> default. This patch needs to be applied after the 
> breakindent_patch.diff.
> 
> 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:
> ,----
> | Lorem ipsum dolor sit amet,
> | consetetur sadipscing elitr,
> | sed
> |     Lorem ipsum dolor sit amet,
> |>>   consetetur sadipscing elitr,
> |>>   sed
> `----
> 
> Any opinions on where the 'sbr' should be drawn (should this be put into 
> another option)?
> 
> Please test this patch and report problems. 
> 
> Bram, would you be willing to include this feature?

I already sent another updated breakindent patch which also fixes
coding style:
https://groups.google.com/d/msg/vim_dev/SML3mtGd50s/VDvFN11MwcEJ
(And minor updated version is available at here:
https://bitbucket.org/k_takata/vim-ktakata-mq/src/f8c3f9f5de704bc74a9d61fc633ecd2266b10d0a/vim-7.4.178-breakindent.patch)

Could you merge it?

Regards,
Ken Takata

-- 
-- 
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 [email protected].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui