Hi Bram,

On Mon, Nov 15, 2021 at 11:40 AM Bram Moolenaar <[email protected]>
wrote:

> Finally had time to look into the patch. It does make sense, I think we
> can use his for all options that take the name of a function as an argument.
> The use of function() and funcref() is not documented. We should make a
> separate section in the help, with some examples, and link to it from the
> options that support this format.
> We have to make sure that the callback is cleared when exiting. The asan
> build doesn't report a leak, but that's probably because the tests end with
> making the option empty.
>
>
>
I have updated the PR to incorporate your comments.

Regards,
Yegappan

-- 
-- 
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].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/CAAW7x7kn0BwgQOTY5B6Az2gC4mGNwV4n5d6%3Dbkt77Fpp7Rgv9Q%40mail.gmail.com.

Raspunde prin e-mail lui