Skip to content

Commit 44c0e13

Browse files
XiretzaXiretza
committed
Convert uses of BuiltinLintDiag::Normal to custom variants
This ensures all diagnostic messages are created at diagnostic emission time, making them translatable.
1 parent 2dcda16 commit 44c0e13

File tree

21 files changed

+267
-148
lines changed

21 files changed

+267
-148
lines changed

‎compiler/rustc_builtin_macros/messages.ftl

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,5 +241,3 @@ builtin_macros_unexpected_lit = expected path to a trait, found literal
241241
.label = not a trait
242242
.str_lit = try using `#[derive({$sym})]`
243243
.other = for example, write `#[derive(Debug)]` for `Debug`
244-
245-
builtin_macros_unnameable_test_items = cannot test inner items

‎compiler/rustc_builtin_macros/src/asm.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
use ast::token::IdentIsRaw;
2+
use lint::BuiltinLintDiag;
23
use rustc_ast as ast;
34
use rustc_ast::ptr::P;
45
use rustc_ast::token::{self, Delimiter};
@@ -509,19 +510,19 @@ fn expand_preparsed_asm(
509510
};
510511

511512
if template_str.contains(".intel_syntax") {
512-
ecx.psess().buffer_lint(
513+
ecx.psess().buffer_lint_with_diagnostic(
513514
lint::builtin::BAD_ASM_STYLE,
514515
find_span(".intel_syntax"),
515516
ecx.current_expansion.lint_node_id,
516-
"avoid using `.intel_syntax`, Intel syntax is the default",
517+
BuiltinLintDiag::AvoidUsingIntelSyntax,
517518
);
518519
}
519520
if template_str.contains(".att_syntax") {
520-
ecx.psess().buffer_lint(
521+
ecx.psess().buffer_lint_with_diagnostic(
521522
lint::builtin::BAD_ASM_STYLE,
522523
find_span(".att_syntax"),
523524
ecx.current_expansion.lint_node_id,
524-
"avoid using `.att_syntax`, prefer using `options(att_syntax)` instead",
525+
BuiltinLintDiag::AvoidUsingAttSyntax,
525526
);
526527
}
527528
}

‎compiler/rustc_builtin_macros/src/source_util.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ use rustc_expand::base::{
1111
use rustc_expand::base::{DummyResult, ExpandResult, ExtCtxt};
1212
use rustc_expand::base::{MacEager, MacResult, MacroExpanderResult};
1313
use rustc_expand::module::DirOwnership;
14+
use rustc_lint_defs::BuiltinLintDiag;
1415
use rustc_parse::new_parser_from_file;
1516
use rustc_parse::parser::{ForceCollect, Parser};
1617
use rustc_session::lint::builtin::INCOMPLETE_INCLUDE;
@@ -143,11 +144,11 @@ pub fn expand_include<'cx>(
143144
fn make_expr(mut self: Box<ExpandInclude<'a>>) -> Option<P<ast::Expr>> {
144145
let expr = parse_expr(&mut self.p).ok()?;
145146
if self.p.token != token::Eof {
146-
self.p.psess.buffer_lint(
147+
self.p.psess.buffer_lint_with_diagnostic(
147148
INCOMPLETE_INCLUDE,
148149
self.p.token.span,
149150
self.node_id,
150-
"include macro expected single expression in source",
151+
BuiltinLintDiag::IncompleteInclude,
151152
);
152153
}
153154
Some(expr)

‎compiler/rustc_builtin_macros/src/test_harness.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ use rustc_ast::{attr, ModKind};
99
use rustc_expand::base::{ExtCtxt, ResolverExpand};
1010
use rustc_expand::expand::{AstFragment, ExpansionConfig};
1111
use rustc_feature::Features;
12+
use rustc_lint_defs::BuiltinLintDiag;
1213
use rustc_session::lint::builtin::UNNAMEABLE_TEST_ITEMS;
1314
use rustc_session::Session;
1415
use rustc_span::hygiene::{AstPass, SyntaxContext, Transparency};
@@ -159,11 +160,11 @@ struct InnerItemLinter<'a> {
159160
impl<'a> Visitor<'a> for InnerItemLinter<'_> {
160161
fn visit_item(&mut self, i: &'a ast::Item) {
161162
if let Some(attr) = attr::find_by_name(&i.attrs, sym::rustc_test_marker) {
162-
self.sess.psess.buffer_lint(
163+
self.sess.psess.buffer_lint_with_diagnostic(
163164
UNNAMEABLE_TEST_ITEMS,
164165
attr.span,
165166
i.id,
166-
crate::fluent_generated::builtin_macros_unnameable_test_items,
167+
BuiltinLintDiag::UnnameableTestItems,
167168
);
168169
}
169170
}

‎compiler/rustc_builtin_macros/src/util.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
use rustc_ast::{attr, AttrStyle, Attribute, MetaItem};
22
use rustc_expand::base::{Annotatable, ExtCtxt};
33
use rustc_feature::AttributeTemplate;
4-
use rustc_lint_defs::builtin::DUPLICATE_MACRO_ATTRIBUTES;
4+
use rustc_lint_defs::{builtin::DUPLICATE_MACRO_ATTRIBUTES, BuiltinLintDiag};
55
use rustc_parse::validate_attr;
66
use rustc_span::Symbol;
77

@@ -37,11 +37,11 @@ pub fn warn_on_duplicate_attribute(ecx: &ExtCtxt<'_>, item: &Annotatable, name:
3737
};
3838
if let Some(attrs) = attrs {
3939
if let Some(attr) = attr::find_by_name(attrs, name) {
40-
ecx.psess().buffer_lint(
40+
ecx.psess().buffer_lint_with_diagnostic(
4141
DUPLICATE_MACRO_ATTRIBUTES,
4242
attr.span,
4343
ecx.current_expansion.lint_node_id,
44-
"duplicated attribute",
44+
BuiltinLintDiag::DuplicateMacroAttribute,
4545
);
4646
}
4747
}

‎compiler/rustc_expand/messages.ftl

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,21 +21,12 @@ expand_attributes_wrong_form =
2121
expand_cannot_be_name_of_macro =
2222
`{$trait_ident}` cannot be a name of {$macro_type} macro
2323
24-
expand_cfg_attr_no_attributes =
25-
`#[cfg_attr]` does not expand to any attributes
26-
2724
expand_collapse_debuginfo_illegal =
2825
illegal value for attribute #[collapse_debuginfo(no|external|yes)]
2926
3027
expand_count_repetition_misplaced =
3128
`count` can not be placed inside the inner-most repetition
3229
33-
expand_crate_name_in_cfg_attr_deprecated =
34-
`crate_name` within an `#![cfg_attr]` attribute is deprecated
35-
36-
expand_crate_type_in_cfg_attr_deprecated =
37-
`crate_type` within an `#![cfg_attr]` attribute is deprecated
38-
3930
expand_custom_attribute_cannot_be_applied =
4031
custom attributes cannot be applied to {$kind ->
4132
[statement] statements

‎compiler/rustc_expand/src/config.rs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ use rustc_attr as attr;
1414
use rustc_data_structures::flat_map_in_place::FlatMapInPlace;
1515
use rustc_feature::Features;
1616
use rustc_feature::{ACCEPTED_FEATURES, REMOVED_FEATURES, UNSTABLE_FEATURES};
17+
use rustc_lint_defs::BuiltinLintDiag;
1718
use rustc_parse::validate_attr;
1819
use rustc_session::parse::feature_err;
1920
use rustc_session::Session;
@@ -248,11 +249,11 @@ impl<'a> StripUnconfigured<'a> {
248249

249250
// Lint on zero attributes in source.
250251
if expanded_attrs.is_empty() {
251-
self.sess.psess.buffer_lint(
252+
self.sess.psess.buffer_lint_with_diagnostic(
252253
rustc_lint_defs::builtin::UNUSED_ATTRIBUTES,
253254
attr.span,
254255
ast::CRATE_NODE_ID,
255-
crate::fluent_generated::expand_cfg_attr_no_attributes,
256+
BuiltinLintDiag::CfgAttrNoAttributes,
256257
);
257258
}
258259

@@ -331,19 +332,19 @@ impl<'a> StripUnconfigured<'a> {
331332
item_span,
332333
);
333334
if attr.has_name(sym::crate_type) {
334-
self.sess.psess.buffer_lint(
335+
self.sess.psess.buffer_lint_with_diagnostic(
335336
rustc_lint_defs::builtin::DEPRECATED_CFG_ATTR_CRATE_TYPE_NAME,
336337
attr.span,
337338
ast::CRATE_NODE_ID,
338-
crate::fluent_generated::expand_crate_type_in_cfg_attr_deprecated,
339+
BuiltinLintDiag::CrateTypeInCfgAttr,
339340
);
340341
}
341342
if attr.has_name(sym::crate_name) {
342-
self.sess.psess.buffer_lint(
343+
self.sess.psess.buffer_lint_with_diagnostic(
343344
rustc_lint_defs::builtin::DEPRECATED_CFG_ATTR_CRATE_TYPE_NAME,
344345
attr.span,
345346
ast::CRATE_NODE_ID,
346-
crate::fluent_generated::expand_crate_name_in_cfg_attr_deprecated,
347+
BuiltinLintDiag::CrateNameInCfgAttr,
347348
);
348349
}
349350
attr

‎compiler/rustc_expand/src/mbe/macro_check.rs

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,8 @@ use crate::mbe::{KleeneToken, TokenTree};
110110
use rustc_ast::token::{Delimiter, IdentIsRaw, Token, TokenKind};
111111
use rustc_ast::{NodeId, DUMMY_NODE_ID};
112112
use rustc_data_structures::fx::FxHashMap;
113-
use rustc_errors::{DiagMessage, MultiSpan};
113+
use rustc_errors::MultiSpan;
114+
use rustc_lint_defs::BuiltinLintDiag;
114115
use rustc_session::lint::builtin::{META_VARIABLE_MISUSE, MISSING_FRAGMENT_SPECIFIER};
115116
use rustc_session::parse::ParseSess;
116117
use rustc_span::symbol::kw;
@@ -252,7 +253,7 @@ fn check_binders(
252253
// 1. The meta-variable is already bound in the current LHS: This is an error.
253254
let mut span = MultiSpan::from_span(span);
254255
span.push_span_label(prev_info.span, "previous declaration");
255-
buffer_lint(psess, span, node_id, "duplicate matcher binding");
256+
buffer_lint(psess, span, node_id, BuiltinLintDiag::DuplicateMatcherBinding);
256257
} else if get_binder_info(macros, binders, name).is_none() {
257258
// 2. The meta-variable is free: This is a binder.
258259
binders.insert(name, BinderInfo { span, ops: ops.into() });
@@ -267,11 +268,11 @@ fn check_binders(
267268
// FIXME: Report this as a hard error eventually and remove equivalent errors from
268269
// `parse_tt_inner` and `nameize`. Until then the error may be reported twice, once
269270
// as a hard error and then once as a buffered lint.
270-
psess.buffer_lint(
271+
psess.buffer_lint_with_diagnostic(
271272
MISSING_FRAGMENT_SPECIFIER,
272273
span,
273274
node_id,
274-
"missing fragment specifier",
275+
BuiltinLintDiag::MissingFragmentSpecifier,
275276
);
276277
}
277278
if !macros.is_empty() {
@@ -595,7 +596,7 @@ fn check_ops_is_prefix(
595596
return;
596597
}
597598
}
598-
buffer_lint(psess, span.into(), node_id, format!("unknown macro variable `{name}`"));
599+
buffer_lint(psess, span.into(), node_id, BuiltinLintDiag::UnknownMacroVariable(name));
599600
}
600601

601602
/// Returns whether `binder_ops` is a prefix of `occurrence_ops`.
@@ -628,30 +629,23 @@ fn ops_is_prefix(
628629
if i >= occurrence_ops.len() {
629630
let mut span = MultiSpan::from_span(span);
630631
span.push_span_label(binder.span, "expected repetition");
631-
let message = format!("variable '{name}' is still repeating at this depth");
632-
buffer_lint(psess, span, node_id, message);
632+
buffer_lint(psess, span, node_id, BuiltinLintDiag::MetaVariableStillRepeating(name));
633633
return;
634634
}
635635
let occurrence = &occurrence_ops[i];
636636
if occurrence.op != binder.op {
637637
let mut span = MultiSpan::from_span(span);
638638
span.push_span_label(binder.span, "expected repetition");
639639
span.push_span_label(occurrence.span, "conflicting repetition");
640-
let message = "meta-variable repeats with different Kleene operator";
641-
buffer_lint(psess, span, node_id, message);
640+
buffer_lint(psess, span, node_id, BuiltinLintDiag::MetaVariableWrongOperator);
642641
return;
643642
}
644643
}
645644
}
646645

647-
fn buffer_lint(
648-
psess: &ParseSess,
649-
span: MultiSpan,
650-
node_id: NodeId,
651-
message: impl Into<DiagMessage>,
652-
) {
646+
fn buffer_lint(psess: &ParseSess, span: MultiSpan, node_id: NodeId, diag: BuiltinLintDiag) {
653647
// Macros loaded from other crates have dummy node ids.
654648
if node_id != DUMMY_NODE_ID {
655-
psess.buffer_lint(META_VARIABLE_MISUSE, span, node_id, message);
649+
psess.buffer_lint_with_diagnostic(META_VARIABLE_MISUSE, span, node_id, diag);
656650
}
657651
}

‎compiler/rustc_interface/src/util.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -389,11 +389,11 @@ pub(crate) fn check_attr_crate_type(
389389
),
390390
);
391391
} else {
392-
lint_buffer.buffer_lint(
392+
lint_buffer.buffer_lint_with_diagnostic(
393393
lint::builtin::UNKNOWN_CRATE_TYPES,
394394
ast::CRATE_NODE_ID,
395395
span,
396-
"invalid `crate_type` value",
396+
BuiltinLintDiag::InvalidCrateTypeValue,
397397
);
398398
}
399399
}

‎compiler/rustc_lint/messages.ftl

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,9 @@ lint_builtin_unused_doc_comment = unused doc comment
161161
lint_builtin_while_true = denote infinite loops with `loop {"{"} ... {"}"}`
162162
.suggestion = use `loop`
163163
164+
lint_cfg_attr_no_attributes =
165+
`#[cfg_attr]` does not expand to any attributes
166+
164167
lint_check_name_unknown_tool = unknown lint tool: `{$tool_name}`
165168
166169
lint_command_line_source = `forbid` lint level was set on command line
@@ -169,6 +172,12 @@ lint_confusable_identifier_pair = found both `{$existing_sym}` and `{$sym}` as i
169172
.current_use = this identifier can be confused with `{$existing_sym}`
170173
.other_use = other identifier used here
171174
175+
lint_crate_name_in_cfg_attr_deprecated =
176+
`crate_name` within an `#![cfg_attr]` attribute is deprecated
177+
178+
lint_crate_type_in_cfg_attr_deprecated =
179+
`crate_type` within an `#![cfg_attr]` attribute is deprecated
180+
172181
lint_cstring_ptr = getting the inner pointer of a temporary `CString`
173182
.as_ptr_label = this pointer will be invalid
174183
.unwrap_label = this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime
@@ -613,6 +622,8 @@ lint_unknown_lint =
613622
lint_unknown_tool_in_scoped_lint = unknown tool name `{$tool_name}` found in scoped lint: `{$tool_name}::{$lint_name}`
614623
.help = add `#![register_tool({$tool_name})]` to the crate root
615624
625+
lint_unnameable_test_items = cannot test inner items
626+
616627
lint_unsupported_group = `{$lint_group}` lint group is not supported with ´--force-warn´
617628
618629
lint_untranslatable_diag = diagnostics should be created using translatable messages
@@ -652,3 +663,6 @@ lint_unused_result = unused result of type `{$ty}`
652663
653664
lint_variant_size_differences =
654665
enum variant is more than three times larger ({$largest} bytes) than the next largest
666+
667+
lint_wasm_c_abi =
668+
older versions of the `wasm-bindgen` crate will be incompatible with future versions of Rust; please update to `wasm-bindgen` v0.2.88

0 commit comments

Comments
 (0)