From 9b144d234e609e87a7bde5854c9748cf77df7eac Mon Sep 17 00:00:00 2001
From: ju6ge <judge@felixrichter.tech>
Date: Thu, 13 Mar 2025 19:58:11 +0100
Subject: [PATCH] optimize log message configuration geneation

---
 rust/src/log/mod.rs | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/rust/src/log/mod.rs b/rust/src/log/mod.rs
index 1f79d53..0832746 100644
--- a/rust/src/log/mod.rs
+++ b/rust/src/log/mod.rs
@@ -1,4 +1,4 @@
-use std::{collections::{BTreeMap, HashMap}, sync::Mutex};
+use std::{collections::HashMap, sync::Mutex};
 use serde::Serialize;
 use strum::{EnumIter, IntoEnumIterator};
 use strum_macros::IntoStaticStr; 
@@ -186,20 +186,24 @@ pub enum LogMessage {
     ConsecutivePumpCountLimit
 }
 
+#[derive(Serialize)]
+pub struct MessageTranslation {
+    msg_type: LogMessage,
+    message: &'static str
+}
+
+impl From<&LogMessage> for MessageTranslation {
+    fn from(value: &LogMessage) -> Self {
+        Self { msg_type: value.clone(), message: value.into() }
+    }
+}
+
 impl LogMessage {
-    pub fn to_log_localisation_config() -> Vec<BTreeMap<&'static str, String>> {
+    pub fn to_log_localisation_config() -> Vec<MessageTranslation> {
         let mut array = Vec::with_capacity(LogMessage::len());
 
         for msg_type in LogMessage::iter() {
-            //todo this should be possible without copy?
-            let name  = msg_type.name().to_owned();
-            let message: &'static str = msg_type.clone().into();
-
-
-            let mut data = BTreeMap::new();
-            data.insert("msg_type", name);
-            data.insert("message", message.to_owned());
-            array[msg_type.ordinal()] = data;
+            array[msg_type.ordinal()] = (&msg_type).into();
         }
         array
     }