Correctly render tab stops in --show-all

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Features
 
+- Correctly render tab stops in --show-all, see #2038 (@Synthetica9)
+
 ## Bugfixes
 
 ## Other

src/preprocessor.rs

@@ -53,26 +53,33 @@ pub fn replace_nonprintable(input: &[u8], tab_width: usize) -> String {
     let tab_width = if tab_width == 0 { 4 } else { tab_width };
 
     let mut idx = 0;
+    let mut line_idx = 0;
     let len = input.len();
     while idx < len {
         if let Some((chr, skip_ahead)) = try_parse_utf8_char(&input[idx..]) {
             idx += skip_ahead;
+            line_idx += 1;
 
             match chr {
                 // space
                 ' ' => output.push('·'),
                 // tab
                 '\t' => {
-                    if tab_width == 1 {
+                    let tab_stop = tab_width - ((line_idx - 1) % tab_width);
+                    line_idx = 0;
+                    if tab_stop == 1 {
                         output.push('↹');
                     } else {
                         output.push('├');
-                        output.push_str(&"─".repeat(tab_width - 2));
+                        output.push_str(&"─".repeat(tab_stop - 2));
                         output.push('┤');
                     }
                 }
                 // line feed
-                '\x0A' => output.push_str("␊\x0A"),
+                '\x0A' => {
+                    output.push_str("␊\x0A");
+                    line_idx = 0;
+                }
                 // carriage return
                 '\x0D' => output.push('␍'),
                 // null

tests/integration_tests.rs

@@ -1217,6 +1217,52 @@ fn show_all_mode() {
         .stderr("");
 }
 
+#[test]
+fn show_all_extends_tab_markers_to_next_tabstop() {
+    bat()
+        .arg("tabs.txt")
+        .arg("--show-all")
+        .arg("--tabs=4")
+        .arg("--style=plain")
+        .assert()
+        .success()
+        .stdout(
+            "├──┤1├─┤2├─┤3├─┤4␊
+1├─┤?␊
+22├┤?␊
+333↹?␊
+4444├──┤?␊
+55555├─┤?␊
+666666├┤?␊
+7777777↹?␊
+88888888├──┤?␊
+",
+        );
+}
+
+#[test]
+fn show_all_extends_tab_markers_to_next_tabstop_width_8() {
+    bat()
+        .arg("tabs.txt")
+        .arg("--show-all")
+        .arg("--tabs=8")
+        .arg("--style=plain")
+        .assert()
+        .success()
+        .stdout(
+            "├──────┤1├─────┤2├─────┤3├─────┤4␊
+1├─────┤?␊
+22├────┤?␊
+333├───┤?␊
+4444├──┤?␊
+55555├─┤?␊
+666666├┤?␊
+7777777↹?␊
+88888888├──────┤?␊
+",
+        );
+}
+
 #[test]
 fn no_paging_arg() {
     bat()