Skip to content

Assorted cleanup in parser & AST validation - #60751

Merged
bors merged 4 commits into
rust-lang:masterfrom
Centril:general-cleanup
May 12, 2019
Merged

Assorted cleanup in parser & AST validation#60751
bors merged 4 commits into
rust-lang:masterfrom
Centril:general-cleanup

Conversation

@Centril

Copy link
Copy Markdown
Contributor

r? @petrochenkov

Extracted out of a larger PR.

};

if op.fixity() == Fixity::None { break }
if let Fixity::None = fixity { break }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ifletFixity::None = fixity{break}
iffixity == Fixity::None{break}

@petrochenkov

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented May 12, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 4aa4a8f has been approved by petrochenkov

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label May 12, 2019
Centril added a commit to Centril/rust that referenced this pull request May 12, 2019
…nkov
Assorted cleanup in parser & AST validation
r? @petrochenkov
Extracted out of a larger PR.
@CentrilCentril mentioned this pull request May 12, 2019
bors added a commit that referenced this pull request May 12, 2019
Rollup of 4 pull requests
Successful merges:
- #60694 (Fix HIR printing of existential type #60662)
- #60750 (syntax: Remove some legacy nonterminal tokens)
- #60751 (Assorted cleanup in parser & AST validation)
- #60752 (Fix minor typos for ItemLocalId)
Failed merges:
r? @ghost
@bors
bors merged commit 4aa4a8f into rust-lang:masterMay 12, 2019
@Centril
Centril deleted the general-cleanup branch May 12, 2019 23:39
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Centril@petrochenkov@bors