Masato Nishihata wrote:

> I'm attaching a document patch that fixes some issues discussed in the
> following link (vim-jp, in Japanese).
> https://github.com/vim-jp/issues/issues/1172
> 
> eval.txt
>     1. Add appendbufline() to short explanation in Builtin Functions list
>     2. Modify an argument name of count() from "list" to "comp" according to
>        the full explanation
>     3. Remove unnecessary text from remote_startserver()
>     4. Modify an argument name of setbufline() from "line" to "text" according
>        to the full explanation
>     5. Add an argument "string" to test_feedinput() according to the full
>        explanation
>     6. Add a link to balloon_eval and balloon_eval_term
>     7. s/setline/getline
>     8. s/VIM/Vim
>     9. s/replace/replaced
>     10. s/dashes/equals
>     11. Modify the explanation of blink in term_getcursor()
>     12. Remove unnecessary "`"
> 
> options.txt
>     1. Move the title of 'conceallevel' option so that it would match with
>        other functions' form
>     2. s/four/five
>     3. s/String/string
> 
> quickfix.txt
>     1. Add *Cfilter* and *Lfilter* (This is not a fix, but just a suggestion.)

Thanks, most changes look good.  I don't like adding a function in the
overview twice, changing the text for appendbufline() makes that work.

Also should do this for append() then.

> There is another discussion about the argument name of append() and
> appendbufline(). It might be better to use the same word like {text} for
> consistency.
> The full explanation of append() shows that {expr} works as list or text, and
> there are two concrete form about that in short explanation ({string} and
> {list}). Meanwhile, {expr} has another meaning (buffer) for appendbufline(),
> and {text} has those meanings instead.

{expr} is more generic, can be anything.  {text} is clearer here, and
can still be a string or a list of strings.

> In short explanation
>     append({lnum}, {string})
>     append({lnum}, {list})
>     appendbufline({expr}, {lnum}, {string}) (added by this patch)
>     appendbufline({expr}, {lnum}, {list}) (added by this patch)
> In full explanation
>     append({lnum}, {expr})
>     appendbufline({expr}, {lnum}, {text})
> 
> Please change or remove if needed. Thank you.

Done.

-- 
hundred-and-one symptoms of being an internet addict:
88. Every single time you press the 'Get mail' button...it does get new mail.

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