Skip to content

Commit 682d748

Browse files
committed
refactor: use named format arguments more often
Especially when there are implicit ones in the same format string.
1 parent 45da448 commit 682d748

File tree

10 files changed

+36
-36
lines changed

10 files changed

+36
-36
lines changed

examples/basic_wio_terminal/src/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -500,8 +500,8 @@ fn button_reporter_task_body() {
500500
if (st ^ new_st) & mask != 0 {
501501
let _ = write!(
502502
Console,
503-
"{b:?}: {}",
504-
["UP", "DOWN"][(new_st & mask != 0) as usize]
503+
"{b:?}: {dir}",
504+
dir = ["UP", "DOWN"][(new_st & mask != 0) as usize]
505505
);
506506
}
507507
}

src/r3_port_arm_m_test_driver/src/board_rp2040.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ fn panic(info: &PanicInfo) -> ! {
2222
// Disable IRQ
2323
cortex_m::interrupt::disable();
2424

25-
r3_support_rp2040::sprintln!("{}{info}", mux::BEGIN_MAIN);
25+
r3_support_rp2040::sprintln!("{BEGIN_MAIN}{info}", BEGIN_MAIN = mux::BEGIN_MAIN);
2626

2727
enter_poll_loop();
2828
}
@@ -50,11 +50,11 @@ impl log::Log for Logger {
5050

5151
fn log(&self, record: &log::Record) {
5252
r3_support_rp2040::sprintln!(
53-
"{}[{:5} {}] {}",
54-
mux::BEGIN_LOG,
55-
record.level(),
56-
record.target(),
57-
record.args()
53+
"{BEGIN_LOG}[{level:5} {target}] {args}",
54+
BEGIN_LOG = mux::BEGIN_LOG,
55+
level = record.level(),
56+
target = record.target(),
57+
args = record.args()
5858
);
5959
}
6060

src/r3_port_arm_m_test_driver/src/logger_rtt.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ impl log::Log for Logger {
1717
if let Some(channel) = &mut *log_channel {
1818
writeln!(
1919
channel,
20-
"[{:5} {}] {}",
21-
record.level(),
22-
record.target(),
23-
record.args()
20+
"[{level:5} {target}] {args}",
21+
level = record.level(),
22+
target = record.target(),
23+
args = record.args()
2424
)
2525
.unwrap();
2626
}

src/r3_port_arm_m_test_driver/src/logger_semihosting.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ impl log::Log for Logger {
77

88
fn log(&self, record: &log::Record) {
99
cortex_m_semihosting::heprintln!(
10-
"[{:5} {}] {}",
11-
record.level(),
12-
record.target(),
13-
record.args()
10+
"[{level:5} {target}] {args}",
11+
level = record.level(),
12+
target = record.target(),
13+
args = record.args()
1414
)
1515
.unwrap();
1616
}

src/r3_port_arm_m_test_driver/src/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@ macro_rules! instantiate_test {
4747

4848
#[cfg(feature = "board-rp_pico")]
4949
r3_support_rp2040::sprintln!(
50-
"{}!- TEST WAS SUCCESSFUL -!",
51-
crate::board_rp2040::mux::BEGIN_MAIN,
50+
"{BEGIN_MAIN}!- TEST WAS SUCCESSFUL -!",
51+
BEGIN_MAIN = crate::board_rp2040::mux::BEGIN_MAIN,
5252
);
5353

5454
#[cfg(feature = "board-rp_pico")]

src/r3_port_arm_test_driver/src/logger_rza1_uart.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ impl log::Log for Logger {
99

1010
fn log(&self, record: &log::Record) {
1111
r3_support_rza1::sprintln!(
12-
"[{:5} {}] {}",
13-
record.level(),
14-
record.target(),
15-
record.args()
12+
"[{level:5} {target}] {args}",
13+
level = record.level(),
14+
target = record.target(),
15+
args = record.args()
1616
);
1717
}
1818

src/r3_port_arm_test_driver/src/logger_semihosting.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ impl log::Log for Logger {
77

88
fn log(&self, record: &log::Record) {
99
arm_semihosting::heprintln!(
10-
"[{:5} {}] {}",
11-
record.level(),
12-
record.target(),
13-
record.args()
10+
"[{level:5} {target}] {args}",
11+
level = record.level(),
12+
target = record.target(),
13+
args = record.args()
1414
)
1515
.unwrap();
1616
}

src/r3_port_riscv_test_driver/src/logger_rtt.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ impl log::Log for Logger {
1717
if let Some(channel) = &mut *log_channel {
1818
writeln!(
1919
channel,
20-
"[{:5} {}] {}",
21-
record.level(),
22-
record.target(),
23-
record.args()
20+
"[{level:5} {target}] {args}",
21+
level = record.level(),
22+
target = record.target(),
23+
args = record.args()
2424
)
2525
.unwrap();
2626
}

src/r3_port_riscv_test_driver/src/logger_uart.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ impl log::Log for Logger {
77

88
fn log(&self, record: &log::Record) {
99
crate::uart::stderr_write_fmt(format_args!(
10-
"[{:5} {}] {}\n",
11-
record.level(),
12-
record.target(),
13-
record.args()
10+
"[{level:5} {target}] {args}\n",
11+
level = record.level(),
12+
target = record.target(),
13+
args = record.args()
1414
));
1515
}
1616

src/r3_test_runner/src/targets/openocd.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,9 @@ impl DebugProbe for GrPeachOpenOcdDebugProbe {
121121
format!(
122122
"{GR_PEACH_INIT}
123123
{GR_PEACH_RESET}
124-
load_image \"{}\"
124+
load_image \"{exe}\"
125125
shutdown",
126-
exe.display(),
126+
exe = exe.display(),
127127
),
128128
)
129129
.await

0 commit comments

Comments
 (0)