From a34064019a381e423bd8b4512725ed941e34fee7 Mon Sep 17 00:00:00 2001
From: Michael Davis <mcarsondavis@gmail.com>
Date: Thu, 10 Apr 2025 14:05:14 -0400
Subject: [PATCH] Remove `Backend::get_cursor`

It is unused and cannot be used on some terminal hosts like `conhost`
that do not respond to VT queries. This doesn't have any affect on
behavior - I'm removing it so that we don't rely on it in the future.
---
 helix-tui/src/backend/mod.rs     |  1 -
 helix-tui/src/backend/termina.rs | 19 -------------------
 helix-tui/src/backend/test.rs    |  4 ----
 helix-tui/src/terminal.rs        |  4 ----
 4 files changed, 28 deletions(-)

diff --git a/helix-tui/src/backend/mod.rs b/helix-tui/src/backend/mod.rs
index 4602f326a..eb5657394 100644
--- a/helix-tui/src/backend/mod.rs
+++ b/helix-tui/src/backend/mod.rs
@@ -21,7 +21,6 @@ pub trait Backend {
         I: Iterator<Item = (u16, u16, &'a Cell)>;
     fn hide_cursor(&mut self) -> Result<(), io::Error>;
     fn show_cursor(&mut self, kind: CursorKind) -> Result<(), io::Error>;
-    fn get_cursor(&mut self) -> Result<(u16, u16), io::Error>;
     fn set_cursor(&mut self, x: u16, y: u16) -> Result<(), io::Error>;
     fn clear(&mut self) -> Result<(), io::Error>;
     fn size(&self) -> Result<Rect, io::Error>;
diff --git a/helix-tui/src/backend/termina.rs b/helix-tui/src/backend/termina.rs
index 5165d7477..b29bbf122 100644
--- a/helix-tui/src/backend/termina.rs
+++ b/helix-tui/src/backend/termina.rs
@@ -511,25 +511,6 @@ impl Backend for TerminaBackend {
         self.flush()
     }
 
-    fn get_cursor(&mut self) -> Result<(u16, u16), io::Error> {
-        write!(
-            self.terminal,
-            "{}",
-            csi::Csi::Cursor(csi::Cursor::RequestActivePositionReport),
-        )?;
-        self.terminal.flush()?;
-        let event = self.terminal.read(|event| {
-            matches!(
-                event,
-                Event::Csi(Csi::Cursor(csi::Cursor::ActivePositionReport { .. }))
-            )
-        })?;
-        let Event::Csi(Csi::Cursor(csi::Cursor::ActivePositionReport { line, col })) = event else {
-            unreachable!();
-        };
-        Ok((line.get_zero_based(), col.get_zero_based()))
-    }
-
     fn set_cursor(&mut self, x: u16, y: u16) -> io::Result<()> {
         let col = OneBased::from_zero_based(x);
         let line = OneBased::from_zero_based(y);
diff --git a/helix-tui/src/backend/test.rs b/helix-tui/src/backend/test.rs
index 47049cd89..37b5ff5ce 100644
--- a/helix-tui/src/backend/test.rs
+++ b/helix-tui/src/backend/test.rs
@@ -139,10 +139,6 @@ impl Backend for TestBackend {
         Ok(())
     }
 
-    fn get_cursor(&mut self) -> Result<(u16, u16), io::Error> {
-        Ok(self.pos)
-    }
-
     fn set_cursor(&mut self, x: u16, y: u16) -> Result<(), io::Error> {
         self.pos = (x, y);
         Ok(())
diff --git a/helix-tui/src/terminal.rs b/helix-tui/src/terminal.rs
index c612cb15c..98e9a7504 100644
--- a/helix-tui/src/terminal.rs
+++ b/helix-tui/src/terminal.rs
@@ -216,10 +216,6 @@ where
         Ok(())
     }
 
-    pub fn get_cursor(&mut self) -> io::Result<(u16, u16)> {
-        self.backend.get_cursor()
-    }
-
     pub fn set_cursor(&mut self, x: u16, y: u16) -> io::Result<()> {
         self.backend.set_cursor(x, y)
     }