If the argument is a list, use its separator in function compact.

utkarshkukreti · utkarshkukreti · commit 8c9547e9a364 · 2014-03-04T19:57:35.000+05:30
diff --git a/functions.cpp b/functions.cpp
@@ -898,6 +898,7 @@ namespace Sass {
     BUILT_IN(compact)
     {
       List* arglist = ARG("$values", List);
+      List::Separator sep = List::COMMA;
       if (arglist->length() == 1) {
         Expression* the_arg = arglist->value_at_index(0);
         arglist = dynamic_cast<List*>(the_arg);
@@ -906,8 +907,9 @@ namespace Sass {
           *result << the_arg;
           return result;
         }
+        sep = arglist->separator();
       }
-      List* result = new (ctx.mem) List(path, position, 0, List::COMMA);
+      List* result = new (ctx.mem) List(path, position, 0, sep);
       for (size_t i = 0, L = arglist->length(); i < L; ++i) {
         Boolean* ith = dynamic_cast<Boolean*>(arglist->value_at_index(i));
         if (ith && ith->value() == false) continue;

Original file line number	Diff line number	Diff line change
`@@ -898,6 +898,7 @@ namespace Sass {`
`898`	`898`	`BUILT_IN(compact)`
`899`	`899`	`{`
`900`	`900`	`List* arglist = ARG("$values", List);`
	`901`	`+ List::Separator sep = List::COMMA;`
`901`	`902`	`if (arglist->length() == 1) {`
`902`	`903`	`Expression* the_arg = arglist->value_at_index(0);`
`903`	`904`	`arglist = dynamic_cast<List*>(the_arg);`
`@@ -906,8 +907,9 @@ namespace Sass {`
`906`	`907`	`*result << the_arg;`
`907`	`908`	`return result;`
`908`	`909`	`}`
	`910`	`+ sep = arglist->separator();`
`909`	`911`	`}`
`910`		`- List* result = new (ctx.mem) List(path, position, 0, List::COMMA);`
	`912`	`+ List* result = new (ctx.mem) List(path, position, 0, sep);`
`911`	`913`	`for (size_t i = 0, L = arglist->length(); i < L; ++i) {`
`912`	`914`	`Boolean* ith = dynamic_cast<Boolean*>(arglist->value_at_index(i));`
`913`	`915`	`if (ith && ith->value() == false) continue;`