On 8 September 2015, Bram Moolenaar <[email protected]> wrote:
> 
> Lcd wrote:
> 
> > On 8 September 2015, Christian Brabandt <[email protected]> wrote:
> > > On Di, 08 Sep 2015, LCD 47 wrote:
> > > 
> > > >     Yes, this finally seems to work.  I suppose the patch below is the
> > > > final version then.
> > > 
> > > Isn't there still something missing?
> > > 
> > > - Still missing the update to the documentation, regarding esc
> > > - Tests for some special <c-k> cases like this one:
> > >   - <c-k>a<esc>cw 
> > 
> >     Does this have anything to do with my initial patch?  I'm afraid
> > I don't understand either the intended behaviour here, or how to
> > test it.
> > 
> > >   - <c-k><space>a
> > >   - <c-k>a: and <c-k>:a
> > 
> >     I believe these are part of the patch?
> >
> > > I think we can ignore the part about special casing certain keys
> > > as mentioned at :h i_CTRL-K That is not reliable testable.
> > >
> > > Personally, I wouldn't create digraphs.vim but use instead this
> > > line in the 2 testfile:
> > >
> > > :if !has("digraphs") | e! test.ok | w! test.out | qa! | endif
> > >
> > > But that is just my personal opinion.
> >
> >     Ok, patch withdrawn.
>
> I don't think we should test all digraphs.  Just testing a few will
> show that the mechanism works.  Doing the whole table doesn't make
> sense, it will just duplicate the list into the tests.

    Sure, but the point was to test digraph listing functionality,
rather than the exact values.  The fact that the values for some common
digraphs (f.i. simple accents) are also tested comes as a bonus. *shrug*

> For the number formats, I think we should only do decimal and hex.
> Octal (with leading zero) often confuses people.

    Using octal numbers for character codes becomes obvious once you
write the latin1 characters in 8x8 tables. :) Admittedly, that's not the
kind of things people tend to know these days though.  Anyway, is there
a reason to disable octal values here while allowing them everywhere
else?

    /lcd

-- 
-- 
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