-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Closed
Labels
A-frontendArea: Compiler frontend (errors, parsing and HIR)Area: Compiler frontend (errors, parsing and HIR)A-parserArea: The lexing & parsing of Rust source code to an ASTArea: The lexing & parsing of Rust source code to an ASTC-bugCategory: This is a bug.Category: This is a bug.P-highHigh priorityHigh priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.
Description
The following is accepted by the parser:
fn foo<T: !'static>() {}
and the !
is just silently dropped thus interpreting the constraint as T: 'static
.
Regression introduced in #57364 on Feb 24.
cc @estebank @hdhoang
I discovered this while working on a refactoring of parser/ty.rs
.
I will fix the issue as part of that refactoring.
YaLTeR, hdhoang, Aaron1011, Ixrec and varkorVeetaha
Metadata
Metadata
Assignees
Labels
A-frontendArea: Compiler frontend (errors, parsing and HIR)Area: Compiler frontend (errors, parsing and HIR)A-parserArea: The lexing & parsing of Rust source code to an ASTArea: The lexing & parsing of Rust source code to an ASTC-bugCategory: This is a bug.Category: This is a bug.P-highHigh priorityHigh priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
Centril commentedon Dec 8, 2019
Fixed in #67148.
Rollup merge of rust-lang#67148 - Centril:ty-polish, r=estebank
Rollup merge of rust-lang#67148 - Centril:ty-polish, r=estebank
Rollup merge of rust-lang#67148 - Centril:ty-polish, r=estebank