From 1b00bbaf2fd772ed35a597feb1ddfa8c0d5d7c0d Mon Sep 17 00:00:00 2001 From: Wez Furlong Date: Thu, 5 May 2022 19:29:19 -0700 Subject: [PATCH] term: remove dead code --- term/src/lib.rs | 2 - term/src/selection.rs | 121 ------------------------------------------ 2 files changed, 123 deletions(-) delete mode 100644 term/src/selection.rs diff --git a/term/src/lib.rs b/term/src/lib.rs index 63451cc57..0b5fdc929 100644 --- a/term/src/lib.rs +++ b/term/src/lib.rs @@ -35,8 +35,6 @@ pub use termwiz::surface::line::*; pub mod screen; pub use crate::screen::*; -pub mod selection; - use termwiz::hyperlink::Hyperlink; pub mod terminal; diff --git a/term/src/selection.rs b/term/src/selection.rs deleted file mode 100644 index 459feaad9..000000000 --- a/term/src/selection.rs +++ /dev/null @@ -1,121 +0,0 @@ -// The range_plus_one lint can't see when the LHS is not compatible with -// and inclusive range -#![cfg_attr(feature = "cargo-clippy", allow(clippy::range_plus_one))] -use super::{ScrollbackOrVisibleRowIndex, VisibleRowIndex}; -#[cfg(feature = "use_serde")] -use serde::{Deserialize, Serialize}; -use std::ops::Range; - -/// The x,y coordinates of either the start or end of a selection region -#[cfg_attr(feature = "use_serde", derive(Deserialize, Serialize))] -#[derive(Debug, Default, Copy, Clone, Eq, PartialEq)] -pub struct SelectionCoordinate { - pub x: usize, - pub y: ScrollbackOrVisibleRowIndex, -} - -/// Represents the selected text range. -/// The end coordinates are inclusive. -#[cfg_attr(feature = "use_serde", derive(Deserialize, Serialize))] -#[derive(Debug, Default, Copy, Clone, Eq, PartialEq)] -pub struct SelectionRange { - pub start: SelectionCoordinate, - pub end: SelectionCoordinate, -} - -impl SelectionRange { - /// Create a new range that starts at the specified location - pub fn start(start: SelectionCoordinate) -> Self { - let end = start; - Self { start, end } - } - - /// Returns a modified version of the selection that is adjusted - /// for a Surface that holds only the visible viewport. - /// The y values are adjusted such that 0 indicates the top of - /// the viewport. - pub fn clip_to_viewport( - &self, - viewport_offset: VisibleRowIndex, - height: usize, - ) -> SelectionRange { - let offset = -viewport_offset as ScrollbackOrVisibleRowIndex; - SelectionRange { - start: SelectionCoordinate { - x: self.start.x, - y: self.start.y.max(offset) - offset, - }, - end: SelectionCoordinate { - x: self.end.x, - y: self - .end - .y - .min(offset + height as ScrollbackOrVisibleRowIndex) - - offset, - }, - } - } - - /// Returns an extended selection that it ends at the specified location - pub fn extend(&self, end: SelectionCoordinate) -> Self { - Self { - start: self.start, - end, - } - } - - /// Return a normalized selection such that the starting y coord - /// is <= the ending y coord. - pub fn normalize(&self) -> Self { - if self.start.y <= self.end.y { - *self - } else { - Self { - start: self.end, - end: self.start, - } - } - } - - /// Yields a range representing the row indices. - /// Make sure that you invoke this on a normalized range! - pub fn rows(&self) -> Range { - debug_assert!( - self.start.y <= self.end.y, - "you forgot to normalize a SelectionRange" - ); - self.start.y..self.end.y + 1 - } - - /// Yields a range representing the selected columns for the specified row. - /// Not that the range may include usize::max_value() for some rows; this - /// indicates that the selection extends to the end of that row. - /// Since this struct has no knowledge of line length, it cannot be - /// more precise than that. - /// Must be called on a normalized range! - pub fn cols_for_row(&self, row: ScrollbackOrVisibleRowIndex) -> Range { - debug_assert!( - self.start.y <= self.end.y, - "you forgot to normalize a SelectionRange" - ); - if row < self.start.y || row > self.end.y { - 0..0 - } else if self.start.y == self.end.y { - // A single line selection - if self.start.x <= self.end.x { - self.start.x..self.end.x.saturating_add(1) - } else { - self.end.x..self.start.x.saturating_add(1) - } - } else if row == self.end.y { - // last line of multi-line - 0..self.end.x.saturating_add(1) - } else if row == self.start.y { - // first line of multi-line - self.start.x..usize::max_value() - } else { - // some "middle" line of multi-line - 0..usize::max_value() - } - } -}