From fbdcd191a9c7b317601d75244b27173774e36b66 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jana=20D=C3=B6nszelmann?= Date: Mon, 6 Apr 2026 17:31:03 +0200 Subject: [PATCH] make `expected_literal` positive --- compiler/rustc_attr_parsing/src/attributes/deprecation.rs | 2 +- compiler/rustc_attr_parsing/src/attributes/link_attrs.rs | 2 +- compiler/rustc_attr_parsing/src/attributes/macro_attrs.rs | 2 +- .../rustc_attr_parsing/src/attributes/proc_macro_attrs.rs | 4 ++-- compiler/rustc_attr_parsing/src/attributes/rustc_internal.rs | 2 +- compiler/rustc_attr_parsing/src/attributes/stability.rs | 4 ++-- compiler/rustc_attr_parsing/src/attributes/test_attrs.rs | 2 +- compiler/rustc_attr_parsing/src/attributes/traits.rs | 2 +- compiler/rustc_attr_parsing/src/context.rs | 4 ++-- compiler/rustc_attr_parsing/src/session_diagnostics.rs | 4 ++-- 10 files changed, 14 insertions(+), 14 deletions(-) diff --git a/compiler/rustc_attr_parsing/src/attributes/deprecation.rs b/compiler/rustc_attr_parsing/src/attributes/deprecation.rs index e9281179a818..35996904e8c7 100644 --- a/compiler/rustc_attr_parsing/src/attributes/deprecation.rs +++ b/compiler/rustc_attr_parsing/src/attributes/deprecation.rs @@ -83,7 +83,7 @@ fn convert(cx: &mut AcceptContext<'_, '_, S>, args: &ArgParser) -> Option { for param in list.mixed() { let Some(param) = param.meta_item() else { - cx.adcx().unexpected_literal(param.span()); + cx.adcx().expected_not_literal(param.span()); return None; }; diff --git a/compiler/rustc_attr_parsing/src/attributes/link_attrs.rs b/compiler/rustc_attr_parsing/src/attributes/link_attrs.rs index e1ae2068f9fd..6ae4e31af9df 100644 --- a/compiler/rustc_attr_parsing/src/attributes/link_attrs.rs +++ b/compiler/rustc_attr_parsing/src/attributes/link_attrs.rs @@ -93,7 +93,7 @@ fn extend( let mut import_name_type = None; for item in items.mixed() { let Some(item) = item.meta_item() else { - cx.adcx().unexpected_literal(item.span()); + cx.adcx().expected_not_literal(item.span()); continue; }; diff --git a/compiler/rustc_attr_parsing/src/attributes/macro_attrs.rs b/compiler/rustc_attr_parsing/src/attributes/macro_attrs.rs index e49ad5695c12..3467d5712e1a 100644 --- a/compiler/rustc_attr_parsing/src/attributes/macro_attrs.rs +++ b/compiler/rustc_attr_parsing/src/attributes/macro_attrs.rs @@ -185,7 +185,7 @@ fn convert(cx: &mut AcceptContext<'_, '_, S>, args: &ArgParser) -> Option( return None; }; let Some(trait_attr) = trait_attr.meta_item() else { - cx.adcx().unexpected_literal(trait_attr.span()); + cx.adcx().expected_not_literal(trait_attr.span()); return None; }; let Some(trait_ident) = trait_attr.path().word() else { @@ -98,7 +98,7 @@ fn parse_derive_like( let mut attributes = ThinVec::new(); if let Some(attrs) = items.next() { let Some(attr_list) = attrs.meta_item() else { - cx.adcx().unexpected_literal(attrs.span()); + cx.adcx().expected_not_literal(attrs.span()); return None; }; if !attr_list.path().word_is(sym::attributes) { diff --git a/compiler/rustc_attr_parsing/src/attributes/rustc_internal.rs b/compiler/rustc_attr_parsing/src/attributes/rustc_internal.rs index 2ff059928d8c..d77c804af697 100644 --- a/compiler/rustc_attr_parsing/src/attributes/rustc_internal.rs +++ b/compiler/rustc_attr_parsing/src/attributes/rustc_internal.rs @@ -744,7 +744,7 @@ fn extend( let mut result = Vec::new(); for item in items.mixed() { let Some(arg) = item.meta_item() else { - cx.adcx().unexpected_literal(item.span()); + cx.adcx().expected_not_literal(item.span()); continue; }; let Some(ident) = arg.ident() else { diff --git a/compiler/rustc_attr_parsing/src/attributes/stability.rs b/compiler/rustc_attr_parsing/src/attributes/stability.rs index 6efcdd1e1b8d..866b53e4c0d9 100644 --- a/compiler/rustc_attr_parsing/src/attributes/stability.rs +++ b/compiler/rustc_attr_parsing/src/attributes/stability.rs @@ -319,7 +319,7 @@ pub(crate) fn parse_stability( for param in list.mixed() { let param_span = param.span(); let Some(param) = param.meta_item() else { - cx.adcx().unexpected_literal(param.span()); + cx.adcx().expected_not_literal(param.span()); return None; }; @@ -390,7 +390,7 @@ pub(crate) fn parse_unstability( for param in list.mixed() { let Some(param) = param.meta_item() else { - cx.adcx().unexpected_literal(param.span()); + cx.adcx().expected_not_literal(param.span()); return None; }; diff --git a/compiler/rustc_attr_parsing/src/attributes/test_attrs.rs b/compiler/rustc_attr_parsing/src/attributes/test_attrs.rs index 1f50d84869f4..0f7baa941702 100644 --- a/compiler/rustc_attr_parsing/src/attributes/test_attrs.rs +++ b/compiler/rustc_attr_parsing/src/attributes/test_attrs.rs @@ -220,7 +220,7 @@ fn convert(cx: &mut AcceptContext<'_, '_, S>, args: &ArgParser) -> Option, args: &ArgParser) -> Option ErrorGuarante /// An error that should be emitted when a [`MetaItemOrLitParser`](crate::parser::MetaItemOrLitParser) /// was expected *not* to be a literal, but instead a meta item. - pub(crate) fn unexpected_literal(&mut self, span: Span) -> ErrorGuaranteed { - self.emit_parse_error(span, AttributeParseErrorReason::UnexpectedLiteral) + pub(crate) fn expected_not_literal(&mut self, span: Span) -> ErrorGuaranteed { + self.emit_parse_error(span, AttributeParseErrorReason::ExpectedNotLiteral) } pub(crate) fn expected_single_argument(&mut self, span: Span) -> ErrorGuaranteed { diff --git a/compiler/rustc_attr_parsing/src/session_diagnostics.rs b/compiler/rustc_attr_parsing/src/session_diagnostics.rs index ebb217a3a125..5b198813dc43 100644 --- a/compiler/rustc_attr_parsing/src/session_diagnostics.rs +++ b/compiler/rustc_attr_parsing/src/session_diagnostics.rs @@ -565,7 +565,7 @@ pub(crate) enum AttributeParseErrorReason<'a> { }, ExpectedNameValueOrNoArgs, ExpectedNonEmptyStringLiteral, - UnexpectedLiteral, + ExpectedNotLiteral, ExpectedNameValue(Option), DuplicateKey(Symbol), ExpectedSpecificArgument { @@ -799,7 +799,7 @@ fn into_diag(self, dcx: DiagCtxtHandle<'a>, level: Level) -> Diag<'a, G> { diag.span_label(self.span, format!("found `{key}` used as a key more than once")); diag.code(E0538); } - AttributeParseErrorReason::UnexpectedLiteral => { + AttributeParseErrorReason::ExpectedNotLiteral => { diag.span_label(self.span, "didn't expect a literal here"); diag.code(E0565); }