use inline format arguments for unimplemented!

The Clippy lint doesn't check unimplemented!, but I think its failure to
do so is a mistake, since unimplemented! also uses format!.
Therefore, we should use inline format arguments.

https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args

Non-inlined code is slightly more difficult to read and understand, as
it requires arguments to be matched against the format string. The
inlined syntax, where allowed, is simpler.

Author: petertseng

exercises/concept/assembly-line/src/lib.rs

@@ -1,7 +1,7 @@
 pub fn production_rate_per_hour(speed: u8) -> f64 {
-    unimplemented!("calculate hourly production rate at speed: {}", speed)
+    unimplemented!("calculate hourly production rate at speed: {speed}")
 }
 
 pub fn working_items_per_minute(speed: u8) -> u32 {
-    unimplemented!("calculate the amount of working items at speed: {}", speed)
+    unimplemented!("calculate the amount of working items at speed: {speed}")
 }

exercises/concept/csv-builder/src/lib.rs

@@ -14,10 +14,7 @@ impl CsvRecordBuilder {
 
     /// Adds an item to the list separated by a space and a comma.
     pub fn add(&mut self, val: &str) {
-        unimplemented!(
-            "implement the `CsvRecordBuilder::add` method, adding {}",
-            val
-        )
+        unimplemented!("implement the `CsvRecordBuilder::add` method, adding {val}")
     }
 
     /// Consumes the builder and returns the comma separated list

exercises/concept/lucians-luscious-lasagna/src/lib.rs

@@ -4,22 +4,16 @@ pub fn expected_minutes_in_oven() -> i32 {
 
 pub fn remaining_minutes_in_oven(actual_minutes_in_oven: i32) -> i32 {
     unimplemented!(
-        "calculate remaining minutes in oven given actual minutes in oven: {}",
-        actual_minutes_in_oven
+        "calculate remaining minutes in oven given actual minutes in oven: {actual_minutes_in_oven}"
     )
 }
 
 pub fn preparation_time_in_minutes(number_of_layers: i32) -> i32 {
-    unimplemented!(
-        "calculate preparation time in minutes for number of layers: {}",
-        number_of_layers
-    )
+    unimplemented!("calculate preparation time in minutes for number of layers: {number_of_layers}")
 }
 
 pub fn elapsed_time_in_minutes(number_of_layers: i32, actual_minutes_in_oven: i32) -> i32 {
     unimplemented!(
-        "calculate elapsed time in minutes for number of layers {} and actual minutes in oven {}",
-        number_of_layers,
-        actual_minutes_in_oven
+        "calculate elapsed time in minutes for number of layers {number_of_layers} and actual minutes in oven {actual_minutes_in_oven}"
     )
 }

exercises/concept/resistor-color/src/lib.rs

@@ -17,10 +17,7 @@ pub fn color_to_value(color: ResistorColor) -> u32 {
 }
 
 pub fn value_to_color_string(value: u32) -> String {
-    unimplemented!(
-        "convert the value {} into a string representation of color",
-        value
-    )
+    unimplemented!("convert the value {value} into a string representation of color")
 }
 
 pub fn colors() -> Vec<ResistorColor> {

exercises/concept/role-playing-game/src/lib.rs

@@ -10,6 +10,6 @@ impl Player {
     }
 
     pub fn cast_spell(&mut self, mana_cost: u32) -> u32 {
-        unimplemented!("Cast a spell of cost {}", mana_cost)
+        unimplemented!("Cast a spell of cost {mana_cost}")
     }
 }

exercises/concept/rpn-calculator/.docs/instructions.md

@@ -106,8 +106,7 @@ pub enum CalculatorInput {
 
 pub fn evaluate(inputs: &[CalculatorInput]) -> Option<i32> {
     unimplemented!(
-        "Given the inputs: {:?}, evaluate them as though they were a Reverse Polish notation expression",
-        inputs,
+        "Given the inputs: {inputs:?}, evaluate them as though they were a Reverse Polish notation expression"
     );
 }
 ```

exercises/concept/rpn-calculator/src/lib.rs

@@ -9,7 +9,6 @@ pub enum CalculatorInput {
 
 pub fn evaluate(inputs: &[CalculatorInput]) -> Option<i32> {
     unimplemented!(
-        "Given the inputs: {:?}, evaluate them as though they were a Reverse Polish notation expression",
-        inputs,
+        "Given the inputs: {inputs:?}, evaluate them as though they were a Reverse Polish notation expression"
     );
 }

exercises/concept/short-fibonacci/src/lib.rs

@@ -7,7 +7,7 @@ pub fn create_empty() -> Vec<u8> {
 ///
 /// Applications often use buffers when serializing data to send over the network.
 pub fn create_buffer(count: usize) -> Vec<u8> {
-    unimplemented!("create a zeroized buffer of {} bytes", count)
+    unimplemented!("create a zeroized buffer of {count} bytes")
 }
 
 /// Create a vector containing the first five elements of the Fibonacci sequence.

exercises/practice/accumulate/src/lib.rs

@@ -1,4 +1,4 @@
 /// What should the type of _function be?
 pub fn map(input: Vec<i32>, _function: ???) -> Vec<i32> {
-    unimplemented!("Transform input vector {:?} using passed function", input);
+    unimplemented!("Transform input vector {input:?} using passed function");
 }

exercises/practice/acronym/src/lib.rs

@@ -1,3 +1,3 @@
 pub fn abbreviate(phrase: &str) -> String {
-    unimplemented!("Given the phrase '{}', return its acronym", phrase);
+    unimplemented!("Given the phrase '{phrase}', return its acronym");
 }