fix echo debashify

include/struc_helper.hpp

@@ -23,8 +23,9 @@ arg* copy(arg* in);
 variable* copy(variable* in);
 
 // testers
-
 bool arg_has_char(char c, arg* in);
+bool possibly_expands(arg* in);
+bool possibly_expands(arglist* in);
 
 // modifiers
 void force_quotes(arg* in);

src/debashify.cpp

@@ -215,13 +215,26 @@ bool debashify_echo(pipeline* pl)
     {
       delete in->args->args[0];
       in->args->args[0] = new arg("printf");
-      in->args->insert(1, new arg("%s\\ "));
+      if(possibly_expands(in->args->args[2]) )
-      if(newline) // newline: add a newline command at the end
       {
-        brace* br = new brace(new list);
+        in->args->insert(1, new arg("%s\\ "));
-        br->lst->add(new condlist(in));
+        if(newline) // newline: add a newline command at the end
-        br->lst->add(make_condlist("echo"));
+        {
-        pl->cmds[0] = br;
+          brace* br = new brace(new list);
+          br->lst->add(new condlist(in));
+          br->lst->add(make_condlist("echo"));
+          pl->cmds[0] = br;
+        }
+      }
+      else
+      {
+        std::string printfarg="'%s";
+        for(uint32_t i=2; i<in->args->size(); i++)
+          printfarg+=" %s";
+        if(newline)
+          printfarg+="\\n";
+        printfarg+="'";
+        in->args->insert(1, new arg(printfarg));
       }
     }
 

src/struc_helper.cpp

@@ -115,6 +115,22 @@ bool arg_has_char(char c, arg* in)
   return false;
 }
 
+bool possibly_expands(arg* in)
+{
+  for(auto it: in->sa)
+    if( (it->type == _obj::subarg_subshell || it->type == _obj::subarg_variable ) && it->quoted == false)
+      return true;
+  return false;
+}
+
+bool possibly_expands(arglist* in)
+{
+  for(auto it: in->args)
+    if(possibly_expands(it))
+      return true;
+  return false;
+}
+
 // ** CLASS EXTENSIONS ** //
 
 /// GETTERS ///