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 and fixed a few problems.
When starting Vim the default values for the 'briopt' option are not
used.
The documentation says that the default for the minimum is 20, but the
implementation uses a default of 0. A default of 20 is much more
useful, so let's use that. I had to change the tests to compensate.
Now we might as well make the default value empty.
I added TODO items in places where I did not understand the code, mainly
failing to pass in the start of the line, thus computing indent on
somewhere halfway a line. If the pointer doesn't matter we better pass
NULL, so it's clear we do not compute the indent. If it does matter I
suspect there are situations where the column is not computed correctly.
Please have a look at these added TODO items.
--
GUARD #2: Wait a minute -- supposing two swallows carried it together?
GUARD #1: No, they'd have to have it on a line.
GUARD #2: Well, simple! They'd just use a standard creeper!
GUARD #1: What, held under the dorsal guiding feathers?
GUARD #2: Well, why not?
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar -- [email protected] -- 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 [email protected].
For more options, visit https://groups.google.com/d/optout.
