Rollup merge of #49104 - csmoe:semicolon_error, r=petrochenkov
improve error message of inner attribute syntax Fixes #49040
This commit is contained in:
commit
73846dca7b
@ -90,7 +90,7 @@ impl<'a> Parser<'a> {
|
||||
debug!("parse_attribute_with_inner_parse_policy: inner_parse_policy={:?} self.token={:?}",
|
||||
inner_parse_policy,
|
||||
self.token);
|
||||
let (span, path, tokens, mut style) = match self.token {
|
||||
let (span, path, tokens, style) = match self.token {
|
||||
token::Pound => {
|
||||
let lo = self.span;
|
||||
self.bump();
|
||||
@ -129,15 +129,6 @@ impl<'a> Parser<'a> {
|
||||
}
|
||||
};
|
||||
|
||||
if inner_parse_policy == InnerAttributeParsePolicy::Permitted &&
|
||||
self.token == token::Semi {
|
||||
self.bump();
|
||||
self.span_warn(span,
|
||||
"this inner attribute syntax is deprecated. The new syntax is \
|
||||
`#![foo]`, with a bang and no semicolon");
|
||||
style = ast::AttrStyle::Inner;
|
||||
}
|
||||
|
||||
Ok(ast::Attribute {
|
||||
id: attr::mk_attr_id(),
|
||||
style,
|
||||
|
12
src/test/ui/issue-49040.rs
Normal file
12
src/test/ui/issue-49040.rs
Normal file
@ -0,0 +1,12 @@
|
||||
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
|
||||
// file at the top-level directory of this distribution and at
|
||||
// http://rust-lang.org/COPYRIGHT.
|
||||
//
|
||||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
||||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
||||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
||||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
#![allow(unused_variables)]; //~ ERROR expected item, found `;`
|
||||
fn main() {}
|
8
src/test/ui/issue-49040.stderr
Normal file
8
src/test/ui/issue-49040.stderr
Normal file
@ -0,0 +1,8 @@
|
||||
error: expected item, found `;`
|
||||
--> $DIR/issue-49040.rs:11:28
|
||||
|
|
||||
LL | #![allow(unused_variables)]; //~ ERROR expected item, found `;`
|
||||
| ^ help: consider removing this semicolon
|
||||
|
||||
error: aborting due to previous error
|
||||
|
Loading…
Reference in New Issue
Block a user