Evgeni Chasnovski

Results 553 comments of Evgeni Chasnovski

> Regarding `expect.error` and `expect.no_error` indeed the problem is backward compatibility. Hence, I did not wanted to complexify the pull request. > > One way I was thinking was that...

> But we _do_ want to make distinctions in the (extui) message protocol, which plugins should be able to (selectively) subscribe to. The suggested approach is outlined [here](https://github.com/neovim/neovim/issues/35535#issuecomment-3476672519): > Note:...

I am not sure about the intricacies, but `vim.bo[buf_id].buflisted = false` followed by `vim.api.nvim_buf_delete(buf_id, { unload = true })` seems to be what `:bdelete` is described to be. Yes, the...

For completeness, I'll duplicate [this comment](https://github.com/neovim/neovim/pull/33330#issuecomment-2781440361) here: > After actually trying it out, [this comment](https://github.com/neovim/neovim/issues/33314#issuecomment-2780814695) describes redundant steps. The `vim.api.nvim_buf_delete(buf_id, { unload = true })` already both unloads the buffer...

> The point was to actually delete the buffer, something which `bdelete` (equally confusingly) fails to do, but bwipeout succeeds with doing. Confusing naming here is indeed a problem. I...

> IMO strictly defining the API "in terms of" existing ex commands is quite misguided. > > Then, on top of those basic options, we should of course _document_ the...

> IMO, I think the new `nvim_buf_del` can receive several boolean options like `unload`, `unlist` and `wipeout` that old ex command can be implemented with options combinations. Using flags is...

I would also say that if `cmd = ''` would be allowed, then `vim.api.nvim_parse_cmd('', {})` should also be valid. It currently throws error.

> empty cmd is only allowed when there is only a range or only a modifier. That's why you're seeing the error. not sure if allowing empty cmd commands without...

Thanks for the suggestion and kind words! The idea is reasonable, but it highly likely won't be a concise change to what is already the biggest module. There is already...