refator(common): Rename TracingTimer::new_debug.

This patch renames `TracingTiming::new_debug` to `new`. The
documentation claims it sets the log level to `debug` while the `level`
is actually an argument of the constructor. It's then wrong, and the
constructor must be renamed.

Author: Hywan

crates/matrix-sdk-common/src/tracing_timer.rs

@@ -33,7 +33,7 @@ impl std::fmt::Debug for TracingTimer {
 
 impl Drop for TracingTimer {
     fn drop(&mut self) {
-        let message = format!("`{}` finished in {:?}", self.id, self.start.elapsed());
+        let message = format!("{} finished in {:?}", self.id, self.start.elapsed());
 
         let enabled = tracing::level_enabled!(self.level) && {
             let interest = self.callsite.interest();
@@ -62,12 +62,8 @@ impl Drop for TracingTimer {
 }
 
 impl TracingTimer {
-    /// Create a new `TracingTimer` at the `debug` log level.
-    pub fn new_debug(
-        callsite: &'static DefaultCallsite,
-        id: String,
-        level: tracing::Level,
-    ) -> Self {
+    /// Create a new `TracingTimer`.
+    pub fn new(callsite: &'static DefaultCallsite, id: String, level: tracing::Level) -> Self {
         Self { id, callsite, start: Instant::now(), level }
     }
 }
@@ -111,7 +107,7 @@ macro_rules! timer {
             fields: []
         };
 
-        $crate::tracing_timer::TracingTimer::new_debug(&__CALLSITE, $string.into(), $level)
+        $crate::tracing_timer::TracingTimer::new(&__CALLSITE, $string.into(), $level)
     }};
 
     ($string:expr) => {