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.
This commit is contained in:
Kevin Ballard 2014-07-13 18:56:19 -07:00 committed by ridiculousfish
parent 20899f2df9
commit d63db59ade

View file

@ -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;