Cleanup and implications of async_call() usage

[numirias]

Cleanup and implications of async_call() usage

[wookayin]

Thanks for your follow-up. For the first two items, I agree with you. Initially I tried with calling like nvm.async_call(fn, arg1, arg2, kwarg=1) but it was a bit confusing to me. Generally speaking the wrapper style seems more intuitive to me as it looks more like calling a callable (function).

For the latter two items, I'm not quite sure. But I would be happy to help with following them up.

[numirias]

Wrapping the callable makes a lot of sense, but I think I'd prefer lambda expressions because that spares us the extra wrapper function while keeping the code easy to read. E.g. instead of

self._wrap_async(self._buf.clear_highlight)(ERROR_HL_ID)

you'd do

self._vim.async_call(lambda: self._buf.clear_highlight(ERROR_HL_ID))

But either way, it's not a pressing issue.

[wookayin]

I also like the lambda-style way of calling, as it looks more concise. I would be happy with you cleaning up the codes in such a way.