From a639fdb7d89e7acb8ca3e158ecf72c27ad525518 Mon Sep 17 00:00:00 2001 From: Mara Bos Date: Mon, 8 Aug 2022 15:51:14 +0200 Subject: [PATCH] Get rid of named_pos in format_args impl. --- compiler/rustc_builtin_macros/src/format.rs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/compiler/rustc_builtin_macros/src/format.rs b/compiler/rustc_builtin_macros/src/format.rs index d96f24c7bc7..53c13873b10 100644 --- a/compiler/rustc_builtin_macros/src/format.rs +++ b/compiler/rustc_builtin_macros/src/format.rs @@ -1316,8 +1316,6 @@ pub fn expand_preparsed_format_args( .map(|span| fmt_span.from_inner(InnerSpan::new(span.start, span.end))) .collect(); - let named_pos: FxHashSet = names.values().cloned().collect(); - let mut cx = Context { ecx, args, @@ -1388,11 +1386,9 @@ pub fn expand_preparsed_format_args( .enumerate() .filter(|(i, ty)| ty.is_empty() && !cx.count_positions.contains_key(&i)) .map(|(i, _)| { - let msg = if named_pos.contains(&i) { - // named argument + let msg = if cx.args[i].name.is_some() { "named argument never used" } else { - // positional argument "argument never used" }; (cx.args[i].expr.span, msg)