Christian Brabandt wrote:

> Bram,
> here are two small patches:
> 
> 1) edit.patch: fixes a comment. This is so tiny, that it doesn't deserve
>    its own patch number.

That's not right, the sentence starts with "Don't do this".  It's indeed
confusing.  Perhaps this works better:

    /*
     * Handle restarting Insert mode.
     * Don't do this for "CTRL-O ." (repeat an insert): In that case we get
     * here with something in the stuff buffer.
     */
    if (restart_edit != 0 && stuff_empty())


> 2) getchar.patch: makes sure, that typebuf.tb_no_abbr_cnt is only set if
>    the ins_typebuf() function was called with the noremap argument being 
>    non-zero. This actually fixes a problem I see with writing 
>    test_edit.vim  because otherwise using feekdeys('str') might not 
>    expand abbreviations which was unexpected for me.

OK, so that test is going to cover this change?  Then perhaps we should
make this change and add the test at the same time.

Perhaps it's time to include that test now.  It appears to be getting
bigger and bigger.

-- 
hundred-and-one symptoms of being an internet addict:
43. You tell the kids they can't use the computer because "Daddy's got work to
    do" and you don't even have a job.

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

Raspunde prin e-mail lui