refactor(ctl): use rustix::termios::tcgetwinsize instead of ioctl

Co-authored-by: Leonardo Di Giovanna <leonardodigiovanna1@gmail.com>
Signed-off-by: Leonardo Grasso <me@leonardograsso.com>

Author: 2 people

tools/premptictl/Cargo.toml

@@ -16,7 +16,8 @@ serde_json = "1"
 libc = "0.2"
 # `process` enables `kill_process` / `test_kill_process` (safe wrappers
 # around `kill(pid, sig)` and `kill(pid, 0)`).
-rustix = { version = "1", features = ["process"] }
+# `termios` enables `tcgetwinsize` (safe wrapper around `ioctl(TIOCGWINSZ)`).
+rustix = { version = "1", features = ["process", "termios"] }
 
 [target.'cfg(windows)'.dependencies]
 uds_windows = "1.2"

tools/premptictl/src/logs_pretty.rs

@@ -1597,9 +1597,7 @@ pub fn detect_term_size() -> (usize, usize) {
     #[cfg(unix)]
     {
         if cols.is_none() || rows.is_none() {
-            let mut ws: libc::winsize = unsafe { std::mem::zeroed() };
-            let r = unsafe { libc::ioctl(libc::STDOUT_FILENO, libc::TIOCGWINSZ, &mut ws) };
-            if r == 0 {
+            if let Ok(ws) = rustix::termios::tcgetwinsize(std::io::stdout()) {
                 if cols.is_none() && ws.ws_col > 20 {
                     cols = Some(ws.ws_col as usize);
                 }