Ticket #4480: Don't escape safe shell chars and multibyte UTF-8

Don't escape safe shell characters commonly used in paths, such as
'/', '.', '-' and '_'.

Don't escape multibyte UTF-8 characters. Escaping each byte separately
in string assignments doesn't work in tcsh. The previous commit
introduces a regression here: tcsh cannot enter directories whose name
is valid UTF-8 but contains non-alphanumeric UTF-8 characters. It used
to work because printf would glue them together correctly, but we no
longer use printf and command substitution because that breaks newlines.

Signed-off-by: Egmont Koblinger <[email protected]>

Author: egmontkob

src/subshell/common.c

@@ -1438,8 +1438,15 @@ create_cd_command (const char *s)
     {
         n = str_cget_next_char_safe (su);
 
-        if (str_isalnum (su))
+        // tcsh doesn't support defining strings with UTF-8 characters broken down into individual
+        // bytes each escaped in octal, such as $'\303\251' instead of 'é'. So copy all valid
+        // multibyte UTF-8 characters as-is, without escaping; they aren't special shell characters
+        // in any shell and don't need protection.
+        // Also don't escape frequent safe filename characters like '/', '.' and such.
+        if ((unsigned char) su[0] >= 0x80 || g_ascii_isalnum (su[0])
+            || strchr ("/.-_", su[0]) != NULL)
         {
+            // It's a safe character that we copy as-is.
             if (line_length + (n - su) > max_length)
             {
                 // wrap to next physical line
@@ -1453,6 +1460,8 @@ create_cd_command (const char *s)
             line_length += (n - su);
         }
         else
+            // It's a special shell character, or maybe an invalid UTF-8 segment.
+            // Escape each byte separately.
             for (size_t c = 0; c < (size_t) (n - su); c++)
             {
                 if (line_length + escaped_char_len > max_length)