Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.Sign up
Too many parens required in first class module syntax #7216
Original bug ID: 7216
include A(val x);;
Error: Syntax error
include A((val x));;
Error: Unbound value x
Comment author: @garrigue
This is originally designed to avoid any syntactic conflict between the core and module levels.
The second option is much simpler. I include a patch that seems to work.
Comment author: @sliquister
I ran into this issue of having to write double parens again. Any chance this patch can be merged? The patch looks good to me (the error case that's removed is now covered by an error case that was there for parenthesized module expr, if someone is curious), and it works as intended.