berkus
/
topola
mirror of https://codeberg.org/topola/topola.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

egui: store invoker in state instead of reconstructing it each time

This commit is contained in:
Mikolaj Wielgus 2024-05-13 04:28:13 +02:00
parent 7bcb3926d4
commit 8a3bf45fd8
2 changed files with 21 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/autorouter/invoker.rs
View File

@ -56,4 +56,8 @@ impl<R: RulesTrait> Invoker<R> {
pub fn redo(&mut self) {
todo!();
}
pub fn autorouter(&self) -> &Autorouter<R> {
&self.autorouter
}
}

27
src/bin/topola-egui/app.rs
View File

@ -58,7 +58,7 @@ pub struct App {
overlay: Option<Overlay>,
#[serde(skip)]
layout: Option<Arc<Mutex<Layout<DsnRules>>>>,
invoker: Option<Arc<Mutex<Invoker<DsnRules>>>>,
#[serde(skip)]
shared_data: Arc<Mutex<SharedData>>,
@ -74,7 +74,7 @@ impl Default for App {
fn default() -> Self {
Self {
overlay: None,
layout: None,
invoker: None,
shared_data: Default::default(),
text_channel: channel(),
from_rect: egui::Rect::from_x_y_ranges(0.0..=1000000.0, 0.0..=500000.0),
@ -157,14 +157,18 @@ impl eframe::App for App {
let design = DsnDesign::load_from_string(file_contents).unwrap();
let layout = design.make_layout();
self.overlay = Some(Overlay::new(&layout).unwrap());
self.layout = Some(Arc::new(Mutex::new(layout)));
self.invoker = Some(Arc::new(Mutex::new(Invoker::new(
Autorouter::new(Arc::new(Mutex::new(layout))).unwrap(),
))));
}
} else {
if let Ok(path) = self.text_channel.1.try_recv() {
let design = DsnDesign::load_from_file(&path).unwrap();
let layout = design.make_layout();
self.overlay = Some(Overlay::new(&layout).unwrap());
self.layout = Some(Arc::new(Mutex::new(layout)));
self.invoker = Some(Arc::new(Mutex::new(Invoker::new(
Autorouter::new(Arc::new(Mutex::new(layout))).unwrap(),
))));
}
}
@ -199,13 +203,14 @@ impl eframe::App for App {
ui.separator();
if ui.button("Autoroute").clicked() {
if let (Some(layout_arc_mutex), Some(overlay)) = (&self.layout, &self.overlay) {
let layout = layout_arc_mutex.clone();
if let (Some(invoker_arc_mutex), Some(overlay)) = (&self.invoker, &self.overlay)
{
let invoker = invoker_arc_mutex.clone();
let shared_data_arc_mutex = self.shared_data.clone();
let selection = overlay.selection().clone();
execute(async move {
let mut invoker = Invoker::new(Autorouter::new(layout).unwrap());
let mut invoker = invoker.lock().unwrap();
let mut execute = invoker.execute_walk(&Command::Autoroute(selection));
if let Execute::Autoroute(ref mut autoroute) = execute {
@ -272,12 +277,14 @@ impl eframe::App for App {
let transform = egui::emath::RectTransform::from_to(self.from_rect, viewport_rect);
let mut painter = Painter::new(ui, transform);
if let Some(layout_arc_mutex) = &self.layout {
if let (layout, shared_data, Some(overlay)) = (
&layout_arc_mutex.lock().unwrap(),
if let Some(invoker_arc_mutex) = &self.invoker {
if let (invoker, shared_data, Some(overlay)) = (
&invoker_arc_mutex.lock().unwrap(),
self.shared_data.lock().unwrap(),
&mut self.overlay,
) {
let layout = &invoker.autorouter().layout().lock().unwrap();
if ctx.input(|i| i.pointer.any_click()) {
overlay.click(
layout,