diff options
| author | Urgau <urgau@numericable.fr> | 2024-01-13 12:18:51 +0100 |
|---|---|---|
| committer | Urgau <urgau@numericable.fr> | 2024-01-13 17:19:46 +0100 |
| commit | 41b69aae912aa9a1a8d5ed9518e73d136b42e7ca (patch) | |
| tree | c0d6cc9db279305d70497c8c45e95a553a857910 /compiler/rustc_lint/src/context/diagnostics.rs | |
| parent | 284cb714d2bf54b1af41dbc18469b478bc6f6489 (diff) | |
| download | rust-41b69aae912aa9a1a8d5ed9518e73d136b42e7ca.tar.gz rust-41b69aae912aa9a1a8d5ed9518e73d136b42e7ca.zip | |
Add way to express no-values with check-cfg
Diffstat (limited to 'compiler/rustc_lint/src/context/diagnostics.rs')
| -rw-r--r-- | compiler/rustc_lint/src/context/diagnostics.rs | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/compiler/rustc_lint/src/context/diagnostics.rs b/compiler/rustc_lint/src/context/diagnostics.rs index 9e6a6f70eac..312874db3f5 100644 --- a/compiler/rustc_lint/src/context/diagnostics.rs +++ b/compiler/rustc_lint/src/context/diagnostics.rs @@ -354,6 +354,15 @@ pub(super) fn builtin( Applicability::MaybeIncorrect, ); } + } else { + db.note(format!("no expected values for `{name}`")); + + let sp = if let Some((_value, value_span)) = value { + name_span.to(value_span) + } else { + name_span + }; + db.span_suggestion(sp, "remove the condition", "", Applicability::MaybeIncorrect); } // We don't want to suggest adding values to well known names @@ -373,6 +382,8 @@ pub(super) fn builtin( if name == sym::feature { if let Some((value, _value_span)) = value { db.help(format!("consider adding `{value}` as a feature in `Cargo.toml`")); + } else { + db.help("consider defining some features in `Cargo.toml`"); } } else if !is_cfg_a_well_know_name { db.help(format!("consider using a Cargo feature instead or adding `println!(\"cargo:rustc-check-cfg={inst}\");` to the top of a `build.rs`")); |
