fix: Lint (#547)

Author: loewenheim

sentry-actix/examples/basic.rs

@@ -39,7 +39,7 @@ fn main() -> io::Result<()> {
     runtime.block_on(async move {
         let addr = "127.0.0.1:3001";
 
-        println!("Starting server on http://{}", addr);
+        println!("Starting server on http://{addr}");
 
         HttpServer::new(|| {
             App::new()

sentry-anyhow/src/lib.rs

@@ -72,7 +72,7 @@ pub fn event_from_error(err: &anyhow::Error) -> Event<'static> {
         // exception records are sorted in reverse
         if let Some(exc) = event.exception.iter_mut().last() {
             let backtrace = err.backtrace();
-            exc.stacktrace = sentry_backtrace::parse_stacktrace(&format!("{:#}", backtrace));
+            exc.stacktrace = sentry_backtrace::parse_stacktrace(&format!("{backtrace:#}"));
         }
     }
 

sentry-contexts/build.rs

@@ -26,7 +26,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
         f,
         "pub const RUSTC_VERSION: Option<&'static str> = {};",
         if let Ok(version) = version() {
-            format!("Some(\"{}\")", version)
+            format!("Some(\"{version}\")")
         } else {
             "None".into()
         }
@@ -45,7 +45,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
                 Channel::Beta => "beta",
                 Channel::Stable => "stable",
             };
-            format!("Some(\"{}\")", chan)
+            format!("Some(\"{chan}\")")
         } else {
             "None".into()
         }
@@ -54,11 +54,10 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     writeln!(f, "/// The platform identifier.")?;
     writeln!(
         f,
-        "#[allow(unused)] pub const PLATFORM: &str = \"{}\";",
-        platform
+        "#[allow(unused)] pub const PLATFORM: &str = \"{platform}\";"
     )?;
     writeln!(f, "/// The CPU architecture identifier.")?;
-    writeln!(f, "pub const ARCH: &str = \"{}\";", arch)?;
+    writeln!(f, "pub const ARCH: &str = \"{arch}\";")?;
 
     println!("cargo:rerun-if-changed=build.rs");
     println!("cargo:rerun-if-changed=Cargo.toml");

sentry-core/src/error.rs

@@ -97,7 +97,7 @@ pub fn event_from_error<E: Error + ?Sized>(err: &E) -> Event<'static> {
 }
 
 fn exception_from_error<E: Error + ?Sized>(err: &E) -> Exception {
-    let dbg = format!("{:?}", err);
+    let dbg = format!("{err:?}");
     let value = err.to_string();
 
     // A generic `anyhow::msg` will just `Debug::fmt` the `String` that you feed
@@ -106,7 +106,7 @@ fn exception_from_error<E: Error + ?Sized>(err: &E) -> Exception {
     // To work around this, we check if the `Debug::fmt` of the complete error
     // matches its `Display::fmt`, in which case there is no type to parse and
     // we will just be using `Error`.
-    let ty = if dbg == format!("{:?}", value) {
+    let ty = if dbg == format!("{value:?}") {
         String::from("Error")
     } else {
         parse_type_from_debug(&dbg).to_owned()
@@ -140,7 +140,7 @@ fn test_parse_type_from_debug() {
     use parse_type_from_debug as parse;
     #[derive(Debug)]
     struct MyStruct;
-    let err = format!("{:?}", MyStruct);
+    let err = format!("{MyStruct:?}");
     assert_eq!(parse(&err), "MyStruct");
 
     let err = format!("{:?}", "NaN".parse::<usize>().unwrap_err());

sentry-core/src/performance.rs

@@ -793,7 +793,7 @@ mod tests {
         );
 
         let trace = SentryTrace(Default::default(), Default::default(), None);
-        let parsed = parse_sentry_trace(&format!("{}", trace));
+        let parsed = parse_sentry_trace(&trace.to_string());
         assert_eq!(parsed, Some(trace));
     }
 

sentry-core/src/profiling.rs

@@ -143,7 +143,7 @@ pub fn debug_images() -> Vec<DebugImage> {
                 .unwrap_or_else(|_| "<main>".to_string());
         }
 
-        let code_id = shlib.id().map(|id| CodeId::new(format!("{}", id)));
+        let code_id = shlib.id().map(|id| CodeId::new(id.to_string()));
         let debug_name = shlib.debug_name().map(|n| n.to_string_lossy().to_string());
 
         // For windows, the `virtual_memory_bias` actually returns the real
@@ -271,7 +271,7 @@ fn get_profile_from_report(
                 .into_iter()
                 .map(|address| -> RustFrame {
                     RustFrame {
-                        instruction_addr: format!("{:p}", address),
+                        instruction_addr: format!("{address:p}"),
                     }
                 })
                 .collect(),

sentry-debug-images/src/images.rs

@@ -59,7 +59,7 @@ pub fn debug_images() -> Vec<DebugImage> {
                 .unwrap_or_else(|_| "<main>".to_string());
         }
 
-        let code_id = shlib.id().map(|id| CodeId::new(format!("{}", id)));
+        let code_id = shlib.id().map(|id| CodeId::new(id.to_string()));
         let debug_name = shlib.debug_name().map(|n| n.to_string_lossy().to_string());
 
         // For windows, the `virtual_memory_bias` actually returns the real

sentry-slog/src/converters.rs

@@ -24,7 +24,7 @@ macro_rules! impl_into {
 }
 impl<'a> Serializer for MapSerializer<'a> {
     fn emit_arguments(&mut self, key: Key, val: &fmt::Arguments) -> slog::Result {
-        self.0.insert(key.into(), format!("{}", val).into());
+        self.0.insert(key.into(), val.to_string().into());
         Ok(())
     }
 

sentry-tracing/src/converters.rs

@@ -89,7 +89,7 @@ impl Visit for FieldVisitor {
     }
 
     fn record_debug(&mut self, field: &Field, value: &dyn std::fmt::Debug) {
-        self.record(field, format!("{:?}", value));
+        self.record(field, format!("{value:?}"));
     }
 }
 

sentry-tracing/src/layer.rs

@@ -174,7 +174,7 @@ where
             if target.is_empty() {
                 op.to_string()
             } else {
-                format!("{}::{}", target, op)
+                format!("{target}::{op}")
             }
         });