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

Update to rustc_lexer version 660 #4590

Merged
merged 2 commits into from
May 24, 2020

Conversation

Julian-Wollersberger
Copy link
Contributor

Change the unescape_*() functions to unescape_literal(), to address the canges I made in rust-lang/rust#72047.

I also noticed some outdated FIXMEs.

Change `unescape_*()` to `unescape_literal()`.
* Done at line 243: "Add validation of `crate` keyword not appearing in the middle of the symbol path"
* Already happened: "Remove validation of unterminated literals (it is already implemented in `tokenize()`)"
* Happens in `unescape()`: "Add validation of character literal containing only a single char"
* Missing: "raw string literals and raw byte string literals"
@lnicola
Copy link
Member

lnicola commented May 24, 2020

#4574 includes this, but I'll rebase it over your PR (for the FIXME changes).

@matklad
Copy link
Member

matklad commented May 24, 2020

bors r+

bors bot added a commit that referenced this pull request May 24, 2020
4590: Update to rustc_lexer version 660 r=matklad a=Julian-Wollersberger

Change the `unescape_*()` functions to `unescape_literal()`, to address the canges I made in rust-lang/rust#72047.

I also noticed some outdated FIXMEs.



Co-authored-by: Julian Wollersberger <[email protected]>
@matklad
Copy link
Member

matklad commented May 24, 2020

Not sure what's up with bors....

bors r+

@bors
Copy link
Contributor

bors bot commented May 24, 2020

Already running a review

@matklad
Copy link
Member

matklad commented May 24, 2020

bors r-

@bors
Copy link
Contributor

bors bot commented May 24, 2020

Canceled.

@matklad
Copy link
Member

matklad commented May 24, 2020

bors r+

@matklad matklad merged commit ea3e84f into rust-lang:master May 24, 2020
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.

3 participants