hush: fix yet another fallout

hush: move fixed testsuites out of hush-bugs/*

5 files changed, 12 insertions, 14 deletions

diff --git a/shell/hush.c b/shell/hush.c
index 603e19fe9..82bad664d 100644
--- a/shell/hush.c
+++ b/shell/hush.c
@@ -2691,19 +2691,18 @@ static int expand_vars_to_list(o_string *output, int n, char *arg, char or_mask)
                 arg = ++p;
         } /* end of "while (SPECIAL_VAR_SYMBOL is found) ..." */
 
-        {
-                int len = strlen(arg);
-                if (len) {
-                        o_debug_list("expand_vars_to_list[a]", output, n);
-                        o_addstr(output, arg, len + 1);
-                        o_debug_list("expand_vars_to_list[b]", output, n);
-                } else if (output->length == o_get_last_ptr(output, n)) { /* expansion is empty */
-                        if (!(ored_ch & 0x80)) { /* all vars were not quoted... */
-                                n--;
-                                /* allow to reuse list[n] later without re-growth */
-                                output->has_empty_slot = 1;
-                        }
-                }
+        if (arg[0]) {
+                o_debug_list("expand_vars_to_list[a]", output, n);
+                o_addstr(output, arg, strlen(arg) + 1);
+                o_debug_list("expand_vars_to_list[b]", output, n);
+        } else if (output->length == o_get_last_ptr(output, n) /* expansion is empty */
+         && !(ored_ch & 0x80) /* and all vars were not quoted. */
+        ) {
+                n--;
+                /* allow to reuse list[n] later without re-growth */
+                output->has_empty_slot = 1;
+        } else {
+                o_addchr(output, '\0');
         }
         return n;
 }
diff --git a/shell/hush_test/hush-bugs/tick.right b/shell/hush_test/hush-psubst/tick.right
index 6ed281c75..6ed281c75 100644
--- a/shell/hush_test/hush-bugs/tick.right
+++ b/shell/hush_test/hush-psubst/tick.right
diff --git a/shell/hush_test/hush-bugs/tick.tests b/shell/hush_test/hush-psubst/tick.tests
index 1f749a9cd..1f749a9cd 100755
--- a/shell/hush_test/hush-bugs/tick.tests
+++ b/shell/hush_test/hush-psubst/tick.tests
diff --git a/shell/hush_test/hush-bugs/tick2.right b/shell/hush_test/hush-psubst/tick2.right
index 216c883b8..216c883b8 100644
--- a/shell/hush_test/hush-bugs/tick2.right
+++ b/shell/hush_test/hush-psubst/tick2.right
diff --git a/shell/hush_test/hush-bugs/tick2.tests b/shell/hush_test/hush-psubst/tick2.tests
index dc518057c..db4e944fe 100755
--- a/shell/hush_test/hush-bugs/tick2.tests
+++ b/shell/hush_test/hush-psubst/tick2.tests
@@ -1,4 +1,3 @@
-# Bag: backticks are executed even inside not-taken if
 if false; then
     echo "FOO"
     tmp=`echo BAR >&2`