Issue with doc on constructors

[tatchi]

without params:

Screen.Recording.2023-11-14.at.11.36.06.mov

with params:

Screen.Recording.2023-11-14.at.11.39.57.mov

[voodoos]

This is probably an upstream issue in Merlin.
Which version of OCaml / merlin-lib are you using ?

[tatchi]

This is probably an upstream issue in Merlin. Which version of OCaml / merlin-lib are you using ?

ocaml --version
The OCaml toplevel, version 5.0.0

ocamllsp --version
1.16.2

merlin-lib            4.9-500     Merlin's libraries

[tatchi]

@voodoos should we move this issue to Merlin repo?

Also, would creating a test for this be helpful? After a quick look, it seems that adding something to https://github.com/ocaml/merlin/tree/master/tests/test-dirs/document might be the right place?

[voodoos]

@voodoos should we move this issue to Merlin repo?

Also, would creating a test for this be helpful? After a quick look, it seems that adding something to https://github.com/ocaml/merlin/tree/master/tests/test-dirs/document might be the right place?

Actually, there is already a similar test: https://github.com/mheiber/merlin/blob/master/tests/test-dirs/document/issue1513.t
Having the issue tracked on Merlin's repository and completing that test or adding a new one would be useful indeed :-)

I expect some of these issues to be fixed when moving to OCaml 5.2 which should allow more precise determination of comments.