Jaehwang Jung

This shouldn't be necessary as `client.request` now handles invalid buffers.

maybe https://github.com/neovim/neovim/pull/25195 fixed the issue, but it's not ported to stable.

note: I'll be offline for the next 3 weeks. I think it's working pretty well, but if anyone finds a problem and is interested in fixing it, please feel free...

iiuc helix doesn't do paritial injection: https://github.com/helix-editor/helix/blob/0c81ef73e17a3d45cd6240fd5933ad99b3a60d01/helix-core/src/syntax.rs#L1097 though the idea of constructing lookup table from region to its index on each update is taken from helix. the batching is primarily...

> > the batching is primarily for correcteness of highlights. without it the highlights will ficker when the buffer is shown in multiple windows as the regions created by on_win...

Since the last comment, I have fixed some bugs, refactored, and added some tests. Also, I reorganized the commits: * **perf(treesitter)!: incremental invalidation** * This improves how LanguageTree manages the...

Dropped the batched invalidation commit (https://github.com/tomtomjhj/neovim/commit/fddf77638a8241d24c9250395626b3ad729c495a). > (Also, some docs seem to be garbled?) where?

looks fine to me. this was removed because it is no longer true. ```diff - child languages. The result is list-like if - • this LanguageTree is the root, in...

> [...] maybe we can rethink the parse() API to better handle multiple ranges due to multiple windows, [...] > > Another route (and potentially better) would be to make...

cc @lewis6991 since he worked on the initial version of `set_hl_group()`.