Merge pull request #18 from probe-rs/Don-t-copy-state-for-each-node

Don't copy state for each node
probe-rs · Feb 19, 2024 · 83aeb84 · 83aeb84
2 parents 0585cf5 + 2e27ea0
commit 83aeb84
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/info_node.rs b/src/info_node.rs
@@ -1,3 +1,5 @@
+use std::rc::Rc;
+
 use yew::{
     classes, function_component, html, use_state, Callback, Html, Properties, UseStateHandle,
 };
@@ -12,7 +14,7 @@ use log_viewer::{
 
 #[derive(Clone, PartialEq, Properties)]
 pub struct InfoNodeProps {
-    pub state: State,
+    pub state: Rc<State>,
     pub node_index: usize,
     pub level_filter: UseStateHandle<LevelFilter>,
 }

diff --git a/src/main.rs b/src/main.rs
@@ -6,6 +6,7 @@ mod pill;
 
 use std::{
     collections::{BTreeMap, HashMap},
+    rc::Rc,
     str::FromStr,
 };
 
@@ -321,7 +322,7 @@ fn App() -> Html {
             <button onclick={oncreate} class={classes!["ml-3","my-3", "px-2", "py-1", "border", "border-black"]}>{"Create"}</button>
             <div class="m-3">
                 {match (&*gist, &*state) {
-                    (Ok(_gist), Some(state)) => html!{<InfoNode state={state.clone()} node_index={0} level_filter={level_filter.clone()} />},
+                    (Ok(_gist), Some(state)) => html!{<InfoNode state={Rc::new(state.clone())} node_index={0} level_filter={level_filter.clone()} />},
                     (Err(error), _) => error.to_string().into(),
                     _ => unreachable!()
                 }}