Skip to content
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

Rust: Fix integer literals' delimiters #984

Merged
merged 2 commits into from
Jun 19, 2019
Merged

Conversation

rhysd
Copy link
Contributor

@rhysd rhysd commented Sep 7, 2018

I noticed _ is not lexed as delimiter in integer literals. This PR fixes the point. Additionally, - was considered as delimiter in 0x literal, but it's not correct. I also fixed the point.

example:

123_456_789
0x123_456_789_abc

@rhysd rhysd changed the title Fix integer literals' delimiters Rust: Fix integer literals' delimiters Sep 7, 2018
@pyrmont
Copy link
Contributor

pyrmont commented Jun 19, 2019

@rhysd I'm really sorry this has been outstanding for so long. It looks good but could you add some examples to the Rust visual sample for future reference? The file is in spec/visual/samples/rust.

@pyrmont pyrmont added the author-action The PR has been reviewed but action by the author is needed label Jun 19, 2019
@rhysd
Copy link
Contributor Author

rhysd commented Jun 19, 2019

I added it at f58aef2. Please check the sample is sufficient in terms of reference.

I'm really sorry this has been outstanding for so long.

Please never mind. Thank you for maintaining awesome project.

@pyrmont
Copy link
Contributor

pyrmont commented Jun 19, 2019

Thanks for updating it so quickly! Just waiting for the Travis CI build to complete and then I'll merge :)

@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label Jun 19, 2019
@pyrmont pyrmont merged commit 530761b into rouge-ruby:master Jun 19, 2019
@pyrmont
Copy link
Contributor

pyrmont commented Jun 19, 2019

@rhysd Thanks again for the contribution! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants