Evaluating function args twice because they might be delayed.

Author: Aaron Leung

eval_apply.cpp

@@ -688,16 +688,16 @@ namespace Sass {
       }
     }
     // need to eval twice because some expressions get delayed
-    // for (size_t i = 0, S = args.size(); i < S; ++i) {
-    //   if (args[i].type() != Node::assignment) {
-    //     args[i].should_eval() = true;
-    //     args[i] = eval(args[i], prefix, env, f_env, new_Node, ctx);
-    //   }
-    //   else {
-    //     args[i][1].should_eval() = true;
-    //     args[i][1] = eval(args[i][1], prefix, env, f_env, new_Node, ctx);
-    //   }
-    // }
+    for (size_t i = 0, S = args.size(); i < S; ++i) {
+      if (args[i].type() != Node::assignment) {
+        args[i].should_eval() = true;
+        args[i] = eval(args[i], prefix, env, f_env, new_Node, ctx);
+      }
+      else {
+        args[i][1].should_eval() = true;
+        args[i][1] = eval(args[i][1], prefix, env, f_env, new_Node, ctx);
+      }
+    }
 
     // Create a new environment for the mixin and link it to the appropriate parent
     Environment bindings;
@@ -737,16 +737,16 @@ namespace Sass {
       }
     }
     // need to eval twice because some expressions get delayed
-    // for (size_t i = 0, S = args.size(); i < S; ++i) {
-    //   if (args[i].type() != Node::assignment) {
-    //     args[i].should_eval() = true;
-    //     args[i] = eval(args[i], prefix, env, f_env, new_Node, ctx);
-    //   }
-    //   else {
-    //     args[i][1].should_eval() = true;
-    //     args[i][1] = eval(args[i][1], prefix, env, f_env, new_Node, ctx);
-    //   }
-    // }
+    for (size_t i = 0, S = args.size(); i < S; ++i) {
+      if (args[i].type() != Node::assignment) {
+        args[i].should_eval() = true;
+        args[i] = eval(args[i], prefix, env, f_env, new_Node, ctx);
+      }
+      else {
+        args[i][1].should_eval() = true;
+        args[i][1] = eval(args[i][1], prefix, env, f_env, new_Node, ctx);
+      }
+    }
 
     // bind arguments
     Environment bindings;