Khady / merlin-eldoc

Type and doc on hover for OCaml and Reason in emacs
GNU General Public License v3.0
44 stars 3 forks source link

Broken highlighting for let #33

Open bbatsov opened 2 years ago

bbatsov commented 2 years ago

image

I guess it should either do nothing or highlight the matching in. (which is what happens if you go the to the in itself)

Khady commented 1 year ago

the highlighting of let when you are on the in is probably related to a different mode related to parenthesis or pairs

bbatsov commented 1 year ago

Hmm, how can that be? This doesn't happen when "highlight occurrences" from this package is disabled.

bbatsov commented 1 year ago

Btw, as a general point - I don't think that highlighting occurrences really belongs here, as that has little to do with eldoc.

Khady commented 1 year ago

This is very surprising to me, I didn't expect merlin to return occurrences when the point is on in.

I'm not against moving the highlighting of occurrences in a different mode. But

If you wish to work on that or other improvements to the mode I'm happy to review them and I'm open to add you as a collaborator on the repo.

bbatsov commented 1 year ago

Let's see how much time I'll have for OSS work this year. :-) Given that soon we'll have tree-sitter support in Emacs it will probably be best to just rely on it for things like occurrences, etc. (and would mean that such functionality can be moved to modes like tuareg) There's also going to be some LSP support built-in, which might reduce the need to use merlin directly. There are lots of interesting possibilities on the horizon.