Merge pull request #2218 from mgreter/bugfix/issue-1452

Error when stringifying empty lists for function calls

Author: mgreter

src/eval.cpp

@@ -797,6 +797,11 @@ namespace Sass {
     Env* env = environment();
     if (!env->has(full_name)) {
       if (!env->has("*[f]")) {
+        for (Argument* arg : args->elements()) {
+          if (List* ls = dynamic_cast<List*>(arg->value())) {
+            if (ls->size() == 0) error("() isn't a valid CSS value.", c->pstate());
+          }
+        }
         args = static_cast<Arguments*>(args->perform(this));
         Function_Call* lit = SASS_MEMORY_NEW(ctx.mem, Function_Call,
                                              c->pstate(),

src/extend.cpp

@@ -1971,7 +1971,7 @@ namespace Sass {
                       Wrapped_Selector* cpy_ws = SASS_MEMORY_NEW(ctx.mem, Wrapped_Selector, *ws);
                       CommaSequence_Selector* cpy_ws_sl = SASS_MEMORY_NEW(ctx.mem, CommaSequence_Selector, sl->pstate());
                       // remove parent selectors from inner selector
-                      if (ext_cs->first()) {
+                      if (ext_cs->first() && ext_cs->first()->head()->length() > 0) {
                         Wrapped_Selector* ext_ws = dynamic_cast<Wrapped_Selector*>(ext_cs->first()->head()->first());
                         if (ext_ws/* && ext_cs->length() == 1*/) {
                           CommaSequence_Selector* ws_cs = dynamic_cast<CommaSequence_Selector*>(ext_ws->selector());