remove the scroll_to and scroll_pos methods

they are unnecessary, you can just call the methods on the screen object
directly

2 files changed, 28 insertions, 36 deletions

diff --git a/src/parser.rs b/src/parser.rs
index a18f7e8..0cabb1a 100644
--- a/src/parser.rs
+++ b/src/parser.rs
@@ -36,12 +36,4 @@ impl Parser {
     pub fn screen_mut(&mut self) -> &mut crate::screen::Screen {
         &mut self.screen
     }
-
-    pub fn scroll_pos(&self) -> usize {
-        self.screen.scrollback()
-    }
-
-    pub fn scroll_to(&mut self, idx: usize) {
-        self.screen.set_scrollback(idx);
-    }
 }
diff --git a/tests/scroll.rs b/tests/scroll.rs
index 7c75c92..9abef12 100644
--- a/tests/scroll.rs
+++ b/tests/scroll.rs
@@ -81,76 +81,76 @@ fn scrollback() {
     parser.process(b"\r\n25\r\n26\r\n27\r\n28\r\n29\r\n30");
     assert_eq!(parser.screen().contents(), "7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30");
 
-    parser.scroll_to(0);
-    assert_eq!(parser.scroll_pos(), 0);
+    parser.screen_mut().set_scrollback(0);
+    assert_eq!(parser.screen().scrollback(), 0);
     assert_eq!(parser.screen().contents(), "7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30");
 
-    parser.scroll_to(1);
-    assert_eq!(parser.scroll_pos(), 1);
+    parser.screen_mut().set_scrollback(1);
+    assert_eq!(parser.screen().scrollback(), 1);
     assert_eq!(parser.screen().contents(), "6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29");
 
-    parser.scroll_to(3);
-    assert_eq!(parser.scroll_pos(), 3);
+    parser.screen_mut().set_scrollback(3);
+    assert_eq!(parser.screen().scrollback(), 3);
     assert_eq!(parser.screen().contents(), "4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27");
 
-    parser.scroll_to(6);
-    assert_eq!(parser.scroll_pos(), 6);
+    parser.screen_mut().set_scrollback(6);
+    assert_eq!(parser.screen().scrollback(), 6);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
-    parser.scroll_to(7);
-    assert_eq!(parser.scroll_pos(), 6);
+    parser.screen_mut().set_scrollback(7);
+    assert_eq!(parser.screen().scrollback(), 6);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
-    parser.scroll_to(0);
-    assert_eq!(parser.scroll_pos(), 0);
+    parser.screen_mut().set_scrollback(0);
+    assert_eq!(parser.screen().scrollback(), 0);
     assert_eq!(parser.screen().contents(), "7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30");
 
-    parser.scroll_to(7);
-    assert_eq!(parser.scroll_pos(), 6);
+    parser.screen_mut().set_scrollback(7);
+    assert_eq!(parser.screen().scrollback(), 6);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
     parser.process(b"\r\n31");
-    assert_eq!(parser.scroll_pos(), 7);
+    assert_eq!(parser.screen().scrollback(), 7);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
     parser.process(b"\r\n32");
-    assert_eq!(parser.scroll_pos(), 8);
+    assert_eq!(parser.screen().scrollback(), 8);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
     parser.process(b"\r\n33");
-    assert_eq!(parser.scroll_pos(), 9);
+    assert_eq!(parser.screen().scrollback(), 9);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
     parser.process(b"\r\n34");
-    assert_eq!(parser.scroll_pos(), 10);
+    assert_eq!(parser.screen().scrollback(), 10);
     assert_eq!(parser.screen().contents(), "1\n2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24");
 
     parser.process(b"\r\n35");
-    assert_eq!(parser.scroll_pos(), 10);
+    assert_eq!(parser.screen().scrollback(), 10);
     assert_eq!(parser.screen().contents(), "2\n3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25");
 
     parser.process(b"\r\n36");
-    assert_eq!(parser.scroll_pos(), 10);
+    assert_eq!(parser.screen().scrollback(), 10);
     assert_eq!(parser.screen().contents(), "3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26");
 
-    parser.scroll_to(12);
-    assert_eq!(parser.scroll_pos(), 10);
+    parser.screen_mut().set_scrollback(12);
+    assert_eq!(parser.screen().scrollback(), 10);
     assert_eq!(parser.screen().contents(), "3\n4\n5\n6\n7\n8\n9\n10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26");
 
-    parser.scroll_to(0);
-    assert_eq!(parser.scroll_pos(), 0);
+    parser.screen_mut().set_scrollback(0);
+    assert_eq!(parser.screen().scrollback(), 0);
     assert_eq!(parser.screen().contents(), "13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30\n31\n32\n33\n34\n35\n36");
 
     parser.process(b"\r\n37\r\n38");
-    assert_eq!(parser.scroll_pos(), 0);
+    assert_eq!(parser.screen().scrollback(), 0);
     assert_eq!(parser.screen().contents(), "15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30\n31\n32\n33\n34\n35\n36\n37\n38");
 
-    parser.scroll_to(5);
-    assert_eq!(parser.scroll_pos(), 5);
+    parser.screen_mut().set_scrollback(5);
+    assert_eq!(parser.screen().scrollback(), 5);
     assert_eq!(parser.screen().contents(), "10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30\n31\n32\n33");
 
     parser.process(b"\r\n39\r\n40");
-    assert_eq!(parser.scroll_pos(), 7);
+    assert_eq!(parser.screen().scrollback(), 7);
     assert_eq!(parser.screen().contents(), "10\n11\n12\n13\n14\n15\n16\n17\n18\n19\n20\n21\n22\n23\n24\n25\n26\n27\n28\n29\n30\n31\n32\n33");
 }