-
Notifications
You must be signed in to change notification settings - Fork 230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
optcomp/cppo syntax support? #202
Comments
Hi! |
For cppo, I think it should be quite easy: merlin should ignore all lines starting with blanks From the cppo manual: a cppo directive is a '#' sign placed |
There is a request for cppo support in merlin just now on caml-list by Dmitry Bely. |
I spent some time on this issue. While not too hard to implement, Merlin architecture doesn't suit this case well. |
This is actually a much lower priority for me now that optcomp has been converted with ppx. Would be cool to have this though. |
It's not clear to me that optcomp is really converted to ppx. Isn't it rather the ppx driver provided by jane street that also integrates optcomp processing? My understanding is that it is defined as a custom lexer, so before the proper ppx level, and in particular it is not clear (to me) that it would be supported by Merlin as other ppx are -- but it should also be possible to implement it at the lexer level in Merlin. |
The optcomp mechanism in janestreet's driver is indeed not a ppx and does not work with merlin. |
@def-lkb any update on this? |
Would also love to have this in merlin |
This would be useful. However, just having Merlin ignore |
Not sure how this would even work in theory but it would be nice if merlin at least didn't report them as syntax errors.
The text was updated successfully, but these errors were encountered: