From 63420d918784f27b932359ba4abafe4b70053969 Mon Sep 17 00:00:00 2001 From: kjuulh Date: Fri, 10 May 2024 22:49:30 +0200 Subject: [PATCH] chore: remove unused Signed-off-by: kjuulh --- .../src/components/render_graph.rs | 2 -- .../src/components/render_graph/classic.rs | 3 ++- .../src/components/render_graph/summarize.rs | 23 ++++++++----------- 3 files changed, 11 insertions(+), 17 deletions(-) diff --git a/crates/hyperlog-tui/src/components/render_graph.rs b/crates/hyperlog-tui/src/components/render_graph.rs index da2f149..d3fddb9 100644 --- a/crates/hyperlog-tui/src/components/render_graph.rs +++ b/crates/hyperlog-tui/src/components/render_graph.rs @@ -1,4 +1,2 @@ -use ratatui::prelude::*; - pub mod classic; pub mod summarize; diff --git a/crates/hyperlog-tui/src/components/render_graph/classic.rs b/crates/hyperlog-tui/src/components/render_graph/classic.rs index 680d9e6..9a77e46 100644 --- a/crates/hyperlog-tui/src/components/render_graph/classic.rs +++ b/crates/hyperlog-tui/src/components/render_graph/classic.rs @@ -1,7 +1,8 @@ use ratatui::prelude::*; -use crate::components::movement_graph::{GraphItemType, MovementGraph, MovementGraphItem}; +use crate::components::movement_graph::{GraphItemType, MovementGraph}; +#[allow(dead_code)] pub trait ClassicRenderGraph { fn render_graph(&self, items: &[usize]) -> Vec; fn render_graph_spans(&self, items: &[usize]) -> Vec>; diff --git a/crates/hyperlog-tui/src/components/render_graph/summarize.rs b/crates/hyperlog-tui/src/components/render_graph/summarize.rs index 66f53e3..6cd8fa4 100644 --- a/crates/hyperlog-tui/src/components/render_graph/summarize.rs +++ b/crates/hyperlog-tui/src/components/render_graph/summarize.rs @@ -2,8 +2,8 @@ use crate::components::movement_graph::{GraphItemType, MovementGraph, MovementGr use itertools::Itertools; use ratatui::prelude::*; -const green: Color = Color::Rgb(127, 255, 0); -const orange: Color = Color::Rgb(255, 165, 0); +const GREEN: Color = Color::Rgb(127, 255, 0); +const ORANGE: Color = Color::Rgb(255, 165, 0); pub trait Summarize { fn heading(&self) -> Vec; @@ -29,7 +29,7 @@ impl Summarize for MovementGraphItem { if done { vec![ Span::from("["), - Span::from("x").fg(green), + Span::from("x").fg(GREEN), Span::from("] "), name, ] @@ -46,7 +46,7 @@ impl Summarize for MovementGraphItem { let items = &self.values.items; - let mut items = if items.len() > 2 { + let items = if items.len() > 2 { vec![ items.first().unwrap().heading(), vec![Span::from("...").fg(Color::DarkGray)], @@ -71,7 +71,7 @@ impl Summarize for MovementGraphItem { .into_iter() .map(|h| { if selected { - h.patch_style(Style::new().fg(orange)) + h.patch_style(Style::new().fg(ORANGE)) } else { h } @@ -98,20 +98,15 @@ impl Summarize for MovementGraphItem { } pub trait SummarizeRenderGraph { fn render_graph(&self, items: &[usize]) -> Vec; - fn render_graph_spans(&self, items: &[usize], depth: usize) - -> Vec>; + fn render_graph_spans(&self, items: &[usize]) -> Vec>; } impl SummarizeRenderGraph for MovementGraph { fn render_graph(&self, items: &[usize]) -> Vec { - self.render_graph_spans(items, 0).to_lines() + self.render_graph_spans(items).to_lines() } - fn render_graph_spans( - &self, - items: &[usize], - depth: usize, - ) -> Vec> { + fn render_graph_spans(&self, items: &[usize]) -> Vec> { match items.split_first() { Some((first, rest)) => match self.items.get(*first) { Some(item) => { @@ -135,7 +130,7 @@ impl SummarizeRenderGraph for MovementGraph { let heading = item.heading(); output.push(heading); - let mut next_level = item.values.render_graph_spans(rest, 0); + let mut next_level = item.values.render_graph_spans(rest); for item in next_level.iter_mut() { item.insert(0, Span::raw(" ".repeat(4))); }