[Rust] Replace `log` with `tracing`

- Added more documentation
- Replaced global named logger for plugins, fixing the issue when the CU has multiple (e.g. statically linked demo)
- Simplified some misc code

This is a breaking change, but I believe there is no better time to make it, we cannot continue to use the `log` crate, it is too limited for our needs.

1 files changed, 1 insertions, 27 deletions

diff --git a/rust/src/architecture.rs b/rust/src/architecture.rs
index 1a6253b0..669b84c8 100644
--- a/rust/src/architecture.rs
+++ b/rust/src/architecture.rs
@@ -68,32 +68,6 @@ pub use instruction::*;
 pub use intrinsic::*;
 pub use register::*;
 
-#[macro_export]
-macro_rules! new_id_type {
-    ($name:ident, $inner_type:ty) => {
-        #[derive(std::fmt::Debug, Clone, Copy, PartialEq, Eq, Hash, PartialOrd, Ord)]
-        pub struct $name(pub $inner_type);
-
-        impl From<$inner_type> for $name {
-            fn from(value: $inner_type) -> Self {
-                Self(value)
-            }
-        }
-
-        impl From<$name> for $inner_type {
-            fn from(value: $name) -> Self {
-                value.0
-            }
-        }
-
-        impl std::fmt::Display for $name {
-            fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-                write!(f, "{}", self.0)
-            }
-        }
-    };
-}
-
 /// The [`Architecture`] trait is the backbone of Binary Ninja's analysis capabilities. It tells the
 /// core how to interpret the machine code into LLIL, a generic intermediate representation for
 /// program analysis.
@@ -1852,7 +1826,7 @@ where
                     return expr.index.0;
                 }
             } else {
-                log::warn!(
+                tracing::warn!(
                     "unable to unpack flag write op: {:?} with {} operands",
                     op,
                     operands.len()