nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2024-11-02 15:32:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

sugg: take into count the debug formatting

Browse Source 
Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
This commit is contained in:
Vincenzo Palazzo 2022-08-20 23:29:38 +00:00
parent dd01122b5c
commit 3d8c7d2c0a
3 changed files with 38 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
compiler/rustc_lint/src/context.rs
View File

@ -865,9 +865,14 @@ pub trait LintContext: Sized {
if let Some(positional_arg_to_replace) = position_sp_to_replace {
let name = if is_formatting_arg { named_arg_name + "$" } else { named_arg_name };
let span_to_replace = if let Ok(positional_arg_content) =
self.sess().source_map().span_to_snippet(positional_arg_to_replace) && positional_arg_content.starts_with(":") {
positional_arg_to_replace.shrink_to_lo()
} else {
positional_arg_to_replace
};
db.span_suggestion_verbose(
positional_arg_to_replace,
span_to_replace,
"use the named argument by name to avoid ambiguity",
name,
Applicability::MaybeIncorrect,

10
src/test/ui/suggestions/sugg_with_positional_args_and_debug_fmt.rs Normal file
View File

@ -0,0 +1,10 @@
// When build the suggesttion take in consideration the `:?`
// https://github.com/rust-lang/rust/issues/100648
#![deny(warnings)]
fn main () {
println!("hello {:?}", world = "world");
//~^ ERROR named argument `world` is not used by name
//~| HELP use the named argument by name to avoid ambiguity
//~| SUGGESTION world
}

21
src/test/ui/suggestions/sugg_with_positional_args_and_debug_fmt.stderr Normal file
View File

@ -0,0 +1,21 @@
error: named argument `world` is not used by name
--> $DIR/sugg_with_positional_args_and_debug_fmt.rs:6:28
|
LL | println!("hello {:?}", world = "world");
| ---- ^^^^^ this named argument is referred to by position in formatting string
| |
| this formatting argument uses named argument `world` by position
|
note: the lint level is defined here
--> $DIR/sugg_with_positional_args_and_debug_fmt.rs:3:9
|
LL | #![deny(warnings)]
| ^^^^^^^^
= note: `#[deny(named_arguments_used_positionally)]` implied by `#[deny(warnings)]`
help: use the named argument by name to avoid ambiguity
|
LL | println!("hello {world:?}", world = "world");
| +++++
error: aborting due to previous error