-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add multiline strings #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it need to use its own ComplexToken?
Especially for 3.0 I would need it to use SYMBOL
The problem with SYMBOL is that it is used in places where multiline string is invalid so I wasn't sure if it was a good idea to do that |
SYMBOL can be compiled anywhere, you just have to parse multilined strings in the right place |
Hmm ok let me actually make a SYMBOL only implementation of this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implemented it without a new ComplexToken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see a much simpler and faster way to implement this, can I merge this and then change it?
Sure |
This PR adds multiline stings to clue. Although regular strings can span multiple lines any tab or newline characters are not preserved. This other type of multiline strings preserves those.
The ` symbol is used to denote multiline strings
When this program is run it should output