fix: ensure SUBDIRED positions are printed for multiple directories without -R

This commit fixes an issue where `//SUBDIRED//` positions were not being printed when using `--dired` with multiple directory arguments but without the `-R` (recursive) flag. The SUBDIRED positions are needed whenever directory headings are printed, which occurs with multiple arguments regardless of whether recursion is enabled.

Author: mattsu2020

src/uu/ls/src/dired.rs

@@ -98,6 +98,8 @@ pub fn print_dired_output(
     if !dired.dired_positions.is_empty() {
         print_positions("//DIRED//", &dired.dired_positions);
     }
+    // SUBDIRED is needed whenever directory headings are printed (multiple args or -R),
+    // so don't gate it on config.recursive.
     if !dired.subdired_positions.is_empty() {
         print_positions("//SUBDIRED//", &dired.subdired_positions);
     }

tests/by-util/test_ls.rs

@@ -5205,6 +5205,51 @@ fn test_ls_dired_recursive() {
         .stdout_contains("//DIRED-OPTIONS// --quoting-style");
 }
 
+#[test]
+fn test_ls_dired_subdired_multiple_dirs_non_recursive() {
+    let scene = TestScenario::new(util_name!());
+    let at = &scene.fixtures;
+
+    at.mkdir("dir1");
+    at.mkdir("dir2");
+    at.touch("dir1/a");
+    at.touch("dir2/b");
+
+    let result = scene
+        .ucmd()
+        .arg("--dired")
+        .arg("dir1")
+        .arg("dir2")
+        .succeeds();
+
+    let output = result.stdout_str().to_string();
+    let subdired_line = output
+        .lines()
+        .find(|&line| line.starts_with("//SUBDIRED//"))
+        .unwrap();
+    let positions: Vec<usize> = subdired_line
+        .split_whitespace()
+        .skip(1)
+        .map(|s| s.parse().unwrap())
+        .collect();
+
+    assert_eq!(positions.len() % 2, 0); // Ensure there's an even number of positions
+
+    let dirnames: Vec<String> = positions
+        .chunks(2)
+        .map(|chunk| {
+            let start_pos = chunk[0];
+            let end_pos = chunk[1];
+            String::from_utf8(output.as_bytes()[start_pos..end_pos].to_vec())
+                .unwrap()
+                .trim()
+                .to_string()
+        })
+        .collect();
+
+    assert_eq!(dirnames, vec!["dir1", "dir2"]);
+}
+
 #[test]
 fn test_ls_dired_outputs_parent_offset() {
     let scene = TestScenario::new(util_name!());