fix(quoting): prevent control chars from triggering quotes in NonEscapedShellQuoter

The initial_quoting_with_show_control function was incorrectly treating
control characters as requiring quoting when show_control=true. This
caused test_control_chars to fail in the MinRustV CI check.

For NonEscapedShellQuoter (used in ls --quoting-style=shell):
- Control chars should NOT trigger quoting
- When show_control=false: they become '?' (not special)
- When show_control=true: shown as-is but still don't need quotes
- Only characters in shell_escaped_char_set trigger quoting

This differs from EscapedShellQuoter which uses dollar-quoting for
control characters.

Fixes the test expectation where shell-show mode should NOT wrap
control characters in quotes unless always_quote is set.

Author: naoNao89

src/uucore/src/lib/features/quoting_style/shell_quoter.rs

@@ -398,16 +398,17 @@ fn initial_quoting_with_show_control(
     input: &[u8],
     dirname: bool,
     always_quote: bool,
-    show_control: bool,
+    _show_control: bool,
 ) -> (Quotes, bool) {
-    // Check for control characters FIRST - they require $'...' which only works with single quotes
-    // But only consider them if we're showing them; if hiding, they become '?' which isn't special
-    let has_control_chars = show_control && input.iter().any(|&c| c < 32 || c == 127);
-
-    if has_control_chars
-        || input
-            .iter()
-            .any(|c| shell_escaped_char_set(dirname).contains(c))
+    // For NonEscapedShellQuoter, control chars don't trigger quoting.
+    // When show_control=false, they become '?' which isn't special.
+    // When show_control=true, they're shown as-is but still don't trigger quoting
+    // (unlike EscapedShellQuoter which uses dollar-quoting for them).
+    // Only characters in shell_escaped_char_set trigger quoting.
+
+    if input
+        .iter()
+        .any(|c| shell_escaped_char_set(dirname).contains(c))
     {
         (Quotes::Single, true)
     } else if input.contains(&b'\'') {