Stopped memory leak.

Removed some heap allocations. Split up how we process command vs user completions
squidowl · Jul 5, 2023 · d8a421e · d8a421e
1 parent 5ab2cc2
commit d8a421e
Show file tree

Hide file tree

Showing 2 changed files with 150 additions and 115 deletions.
diff --git a/src/widget/input.rs b/src/widget/input.rs
@@ -74,7 +74,6 @@ where
  type Event = Event;
 
  fn update(&mut self, state: &mut Self::State, event: Self::Event) -> Option<Message> {
- let users = &self.users;
  match event {
  Event::Input(input) => {
  // Reset error state
@@ -84,7 +83,7 @@ where
 
  state.input = input;
 
- state.completion.process(&state.input, users.clone());
+ state.completion.process(&state.input, &self.users[..]);
 
  None
  }
@@ -139,7 +138,7 @@ where
  .get(state.selected_history.unwrap())
  .unwrap()
  .clone();
- state.completion.process(&state.input);
+ state.completion.process(&state.input, &self.users);
 
  return Some(self.on_completion.clone());
  }
@@ -156,7 +155,7 @@ where
  } else {
  *index -= 1;
  state.input = state.history.get(*index).unwrap().clone();
- state.completion.process(&state.input);
+ state.completion.process(&state.input, &self.users);
  }
 
  return Some(self.on_completion.clone());