Implemented hex and octal parsing for row columns for stty

Author: ChrisDryden

src/uu/stty/src/stty.rs

@@ -471,11 +471,20 @@ fn parse_u8_or_err(arg: &str) -> Result<u8, String> {
 
 /// GNU uses an unsigned 32-bit integer for row/col sizes, but then wraps around 16 bits
 /// this function returns Some(n), where n is a u16 row/col size, or None if the string arg cannot be parsed as a u32
+/// Supports decimal, hexadecimal (0x prefix), and octal (0 prefix) formats
 fn parse_rows_cols(arg: &str) -> Option<u16> {
-    if let Ok(n) = arg.parse::<u32>() {
-        return Some((n % (u16::MAX as u32 + 1)) as u16);
-    }
-    None
+    let n = if let Some(hex) = arg.strip_prefix("0x").or_else(|| arg.strip_prefix("0X")) {
+        u32::from_str_radix(hex, 16).ok()?
+    } else if let Some(octal) = arg.strip_prefix('0') {
+        if octal.is_empty() {
+            0
+        } else {
+            u32::from_str_radix(octal, 8).ok()?
+        }
+    } else {
+        arg.parse::<u32>().ok()?
+    };
+    Some((n % (u16::MAX as u32 + 1)) as u16)
 }
 
 fn check_flag_group<T>(flag: &Flag<T>, remove: bool) -> bool {

tests/by-util/test_stty.rs

@@ -282,6 +282,34 @@ fn row_column_sizes() {
         .stderr_contains("missing argument to 'rows'");
 }
 
+#[test]
+#[cfg(unix)]
+fn test_row_column_hex_octal() {
+    let (path, _controller, _replica) = pty_path();
+    let (_at, ts) = at_and_ts!();
+
+    // Test various numeric formats: hex (0x1E), octal (036), uppercase hex (0X1E), decimal (30), and zero
+    let test_cases = [
+        ("rows", "0x1E"),  // hexadecimal = 30
+        ("rows", "036"),   // octal = 30
+        ("cols", "0X1E"),  // uppercase hex = 30
+        ("columns", "30"), // decimal = 30
+        ("rows", "0"),     // zero (not octal prefix)
+    ];
+
+    for (setting, value) in test_cases {
+        let result = ts.ucmd().args(&["--file", &path, setting, value]).run();
+        let exp_result =
+            unwrap_or_return!(expected_result(&ts, &["--file", &path, setting, value]));
+        let normalized_stderr = normalize_stderr(result.stderr_str());
+
+        result
+            .stdout_is(exp_result.stdout_str())
+            .code_is(exp_result.code());
+        assert_eq!(normalized_stderr, exp_result.stderr_str());
+    }
+}
+
 #[test]
 #[cfg(any(target_os = "linux", target_os = "android"))]
 fn line() {