Clean up the IFS handling in command substitution

Remove the useless ASCII test of the first byte of IFS. We don't split
on the first character, we only use a non-empty IFS as a signal to split
on newlines.

1 files changed, 1 insertions, 10 deletions

diff --git a/exec.cpp b/exec.cpp
index f2d168a9..dfd04231 100644
--- a/exec.cpp
+++ b/exec.cpp
@@ -1548,16 +1548,7 @@ static int exec_subshell_internal(const wcstring &cmd, wcstring_list_t *lst, boo
 
     if (! ifs.missing_or_empty())
     {
-        if (ifs.at(0) < 128)
-        {
-            sep = '\n';//ifs[0];
-        }
-        else
-        {
-            sep = 0;
-            debug(0, L"Warning - invalid command substitution separator '%lc'. Please change the first character of IFS", ifs[0]);
-        }
-
+        sep = '\n';
     }
 
     is_subshell=1;