Skip to content

Commit 88e4def

Browse files
committed
Update E0453 to new error format
Fixes rust-lang#35929. Part of rust-lang#35233. r? @jonathandturner
1 parent f5499a0 commit 88e4def

File tree

4 files changed

+15
-7
lines changed

4 files changed

+15
-7
lines changed

src/librustc/lint/context.rs

+3-2
Original file line numberDiff line numberDiff line change
@@ -606,11 +606,12 @@ pub trait LintContext: Sized {
606606
"{}({}) overruled by outer forbid({})",
607607
level.as_str(), lint_name,
608608
lint_name);
609+
diag_builder.span_label(span, &format!("overruled by previous forbid"));
609610
match now_source {
610611
LintSource::Default => &mut diag_builder,
611612
LintSource::Node(forbid_source_span) => {
612-
diag_builder.span_note(forbid_source_span,
613-
"`forbid` lint level set here")
613+
diag_builder.span_label(forbid_source_span,
614+
&format!("`forbid` level set here"))
614615
},
615616
LintSource::CommandLine => {
616617
diag_builder.note("`forbid` lint level was set on command line")

src/test/compile-fail-fulldeps/lint-plugin-forbid-attrs.rs

+4-2
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,13 @@
1515
#![plugin(lint_plugin_test)]
1616
#![forbid(test_lint)]
1717
//~^ NOTE lint level defined here
18-
//~| NOTE `forbid` lint level set here
18+
//~| NOTE `forbid` level set here
1919

2020
fn lintme() { } //~ ERROR item is named 'lintme'
2121

22-
#[allow(test_lint)] //~ ERROR allow(test_lint) overruled by outer forbid(test_lint)
22+
#[allow(test_lint)]
23+
//~^ ERROR allow(test_lint) overruled by outer forbid(test_lint)
24+
//~| NOTE overruled by previous forbid
2325
pub fn main() {
2426
lintme();
2527
}

src/test/compile-fail/E0453.rs

+4-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,10 @@
99
// except according to those terms.
1010

1111
#![forbid(non_snake_case)]
12+
//~^ NOTE `forbid` level set here
1213

13-
#[allow(non_snake_case)] //~ ERROR E0453
14+
#[allow(non_snake_case)]
15+
//~^ ERROR allow(non_snake_case) overruled by outer forbid(non_snake_case)
16+
//~| NOTE overruled by previous forbid
1417
fn main() {
1518
}

src/test/compile-fail/lint-forbid-attr.rs

+4-2
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,10 @@
99
// except according to those terms.
1010

1111
#![forbid(deprecated)]
12-
//~^ NOTE `forbid` lint level set here
12+
//~^ NOTE `forbid` level set here
1313

14-
#[allow(deprecated)] //~ ERROR allow(deprecated) overruled by outer forbid(deprecated)
14+
#[allow(deprecated)]
15+
//~^ ERROR allow(deprecated) overruled by outer forbid(deprecated)
16+
//~| NOTE overruled by previous forbid
1517
fn main() {
1618
}

0 commit comments

Comments
 (0)