Standardize panic message syntax

nautechsystems · Oct 7, 2024 · 296841a · 296841a
1 parent c624480
commit 296841a
Show file tree

Hide file tree

Showing 10 changed files with 16 additions and 27 deletions.
diff --git a/nautilus_core/indicators/src/indicator.rs b/nautilus_core/indicators/src/indicator.rs
@@ -36,31 +36,31 @@ pub trait Indicator {
     fn initialized(&self) -> bool;
 
     fn handle_delta(&mut self, delta: &OrderBookDelta) {
-        panic!("`handle_delta` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_delta` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_deltas(&mut self, deltas: &OrderBookDeltas) {
-        panic!("`handle_deltas` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_deltas` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_depth(&mut self, depth: &OrderBookDepth10) {
-        panic!("`handle_depth` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_depth` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_book(&mut self, book: &OrderBook) {
-        panic!("`handle_book_mbo` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_book_mbo` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_quote(&mut self, quote: &QuoteTick) {
-        panic!("`handle_quote_tick` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_quote_tick` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_trade(&mut self, trade: &TradeTick) {
-        panic!("`handle_trade_tick` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_trade_tick` {IMPL_ERR} `{}`", self.name());
     }
 
     fn handle_bar(&mut self, bar: &Bar) {
-        panic!("`handle_bar` {} `{}`", IMPL_ERR, self.name());
+        panic!("`handle_bar` {IMPL_ERR} `{}`", self.name());
     }
 
     fn reset(&mut self);

diff --git a/nautilus_core/infrastructure/src/sql/models/orders.rs b/nautilus_core/infrastructure/src/sql/models/orders.rs
@@ -105,10 +105,7 @@ impl<'r> FromRow<'r, PgRow> for OrderEventAnyModel {
             let model = OrderUpdatedModel::from_row(row)?;
             Ok(OrderEventAnyModel(OrderEventAny::Updated(model.0)))
         } else {
-            panic!(
-                "Unknown order event kind: {} in Postgres transformation",
-                kind
-            )
+            panic!("Unknown order event kind: {kind} in Postgres transformation",)
         }
     }
 }

diff --git a/nautilus_core/model/src/orders/any.rs b/nautilus_core/model/src/orders/any.rs
@@ -686,9 +686,8 @@ impl AsRef<StopMarketOrder> for OrderAny {
         match self {
             OrderAny::StopMarket(ref order) => order,
             _ => panic!(
-                "Invalid `OrderAny` not `{}`, was {:?}",
+                "Invalid `OrderAny` not `{}`, was {self:?}",
                 stringify!(StopMarketOrder),
-                self
             ),
         }
     }

diff --git a/nautilus_core/model/src/orders/limit.rs b/nautilus_core/model/src/orders/limit.rs
@@ -434,9 +434,8 @@ impl From<OrderAny> for LimitOrder {
         match order {
             OrderAny::Limit(order) => order,
             _ => panic!(
-                "Invalid `OrderAny` not `{}`, was {:?}",
+                "Invalid `OrderAny` not `{}`, was {order:?}",
                 stringify!(LimitOrder),
-                order
             ),
         }
     }

diff --git a/nautilus_core/model/src/orders/limit_if_touched.rs b/nautilus_core/model/src/orders/limit_if_touched.rs
@@ -381,9 +381,8 @@ impl From<OrderAny> for LimitIfTouchedOrder {
             OrderAny::LimitIfTouched(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(LimitIfTouchedOrder),
-                    order
                 )
             }
         }

diff --git a/nautilus_core/model/src/orders/market.rs b/nautilus_core/model/src/orders/market.rs
@@ -455,9 +455,8 @@ impl From<OrderAny> for MarketOrder {
             OrderAny::Market(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(MarketOrder),
-                    order
                 )
             }
         }

diff --git a/nautilus_core/model/src/orders/market_if_touched.rs b/nautilus_core/model/src/orders/market_if_touched.rs
@@ -373,9 +373,8 @@ impl From<OrderAny> for MarketIfTouchedOrder {
             OrderAny::MarketIfTouched(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(MarketIfTouchedOrder),
-                    order
                 )
             }
         }

diff --git a/nautilus_core/model/src/orders/stop_limit.rs b/nautilus_core/model/src/orders/stop_limit.rs
@@ -390,9 +390,8 @@ impl From<OrderAny> for StopLimitOrder {
             OrderAny::StopLimit(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(StopLimitOrder),
-                    order
                 )
             }
         }

diff --git a/nautilus_core/model/src/orders/trailing_stop_limit.rs b/nautilus_core/model/src/orders/trailing_stop_limit.rs
@@ -390,9 +390,8 @@ impl From<OrderAny> for TrailingStopLimitOrder {
             OrderAny::TrailingStopLimit(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(TrailingStopLimitOrder),
-                    order
                 )
             }
         }

diff --git a/nautilus_core/model/src/orders/trailing_stop_market.rs b/nautilus_core/model/src/orders/trailing_stop_market.rs
@@ -380,9 +380,8 @@ impl From<OrderAny> for TrailingStopMarketOrder {
             OrderAny::TrailingStopMarket(order) => order,
             _ => {
                 panic!(
-                    "Invalid `OrderAny` not `{}`, was {:?}",
+                    "Invalid `OrderAny` not `{}`, was {order:?}",
                     stringify!(TrailingStopMarketOrder),
-                    order
                 )
             }
         }