From 0c93cafccff2fb7c5ef786164eade2cf22e99242 Mon Sep 17 00:00:00 2001 From: Thomas Frans Date: Sun, 18 Jun 2023 16:41:47 +0200 Subject: [PATCH] refactor: remove unused `screenchange` variable The `screenchange` member variable on `Layout` didn't seem to be used for anything except for debugging. A comment in `on_layout` makes it seem like it wasn't meant to be used for debugging purposes but for actual functionality that wasn't implemented. --- src/ui/layout.rs | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/ui/layout.rs b/src/ui/layout.rs index 229a11f..76a6236 100644 --- a/src/ui/layout.rs +++ b/src/ui/layout.rs @@ -11,7 +11,6 @@ use cursive::vec::Vec2; use cursive::view::{CannotFocus, IntoBoxedView, Selector}; use cursive::views::EditView; use cursive::{Cursive, Printer}; -use log::debug; use unicode_width::UnicodeWidthStr; use crate::application::UserData; @@ -31,7 +30,6 @@ pub struct Layout { cmdline_focus: bool, result: Result, String>, result_time: Option, - screenchange: bool, last_size: Vec2, ev: events::EventManager, theme: Theme, @@ -100,7 +98,6 @@ impl Layout { cmdline_focus: false, result: Ok(None), result_time: None, - screenchange: true, last_size: Vec2::new(0, 0), ev: ev.clone(), theme, @@ -150,7 +147,6 @@ impl Layout { let s = id.into(); self.focus = Some(s); self.cmdline_focus = false; - self.screenchange = true; // trigger a redraw self.ev.trigger(); @@ -343,13 +339,6 @@ impl View for Layout { if let Some(view) = self.get_current_view_mut() { view.layout(Vec2::new(size.x, size.y - 3)); } - - // the focus view has changed, let the views know so they can redraw - // their items - if self.screenchange { - debug!("layout: new screen selected: {:?}", self.focus); - self.screenchange = false; - } } fn required_size(&mut self, constraint: Vec2) -> Vec2 {