Add support for nested functions in calc()

- Simplify mathOn switch

Author: matthew-dean

dist/less.js

@@ -1003,7 +1003,11 @@ contexts.Eval.prototype.outOfParenthesis = function () {
     this.parensStack.pop();
 };
 
+contexts.Eval.prototype.mathOn = true;
 contexts.Eval.prototype.isMathOn = function () {
+    if (!this.mathOn) {
+        return false;
+    }
     return this.strictMath ? (this.parensStack && this.parensStack.length) : true;
 };
 
@@ -4019,6 +4023,7 @@ var Parser = function Parser(context, imports, fileInfo) {
                     }
 
                     parserInput.forget();
+                    
                     return new(tree.Call)(name, args, index, fileInfo);
                 },
 
@@ -4194,7 +4199,7 @@ var Parser = function Parser(context, imports, fileInfo) {
                     }
                 },
 
-                // A property entity useing the protective {} e.g. @{prop}
+                // A property entity useing the protective {} e.g. ${prop}
                 propertyCurly: function () {
                     var curly, index = parserInput.i;
 
@@ -6331,6 +6336,7 @@ var Node = require("./node"),
 var Call = function (name, args, index, currentFileInfo) {
     this.name = name;
     this.args = args;
+    this.mathOn = name === 'calc' ? false : true;
     this._index = index;
     this._fileInfo = currentFileInfo;
 };
@@ -6353,8 +6359,16 @@ Call.prototype.accept = function (visitor) {
 // The function should receive the value, not the variable.
 //
 Call.prototype.eval = function (context) {
-    var args = this.args.map(function (a) { return a.eval(context); }),
-        result, funcCaller = new FunctionCaller(this.name, context, this.getIndex(), this.fileInfo());
+
+    /**
+     * Turn off math for calc(), and switch back on for evaluating nested functions
+     */
+    var currentMathContext = context.mathOn;
+    context.mathOn = this.mathOn;
+    var args = this.args.map(function (a) { return a.eval(context); });
+    context.mathOn = currentMathContext;
+
+    var result, funcCaller = new FunctionCaller(this.name, context, this.getIndex(), this.fileInfo());
 
     if (funcCaller.isValid()) {
         try {
@@ -8384,7 +8398,6 @@ module.exports = Property;
 
 },{"./declaration":58,"./node":73}],77:[function(require,module,exports){
 var Node = require("./node"),
-    JsEvalNode = require("./js-eval-node"),
     Variable = require("./variable"),
     Property = require("./property");
 
@@ -8441,7 +8454,7 @@ Quoted.prototype.compare = function (other) {
 };
 module.exports = Quoted;
 
-},{"./js-eval-node":67,"./node":73,"./property":76,"./variable":85}],78:[function(require,module,exports){
+},{"./node":73,"./property":76,"./variable":85}],78:[function(require,module,exports){
 var Node = require("./node"),
     Declaration = require("./declaration"),
     Keyword = require("./keyword"),

dist/less.min.js

@@ -421,11 +421,8 @@ var Parser = function Parser(context, imports, fileInfo) {
                     }
 
                     parserInput.forget();
-                    var call = new(tree.Call)(name, args, index, fileInfo);
-                    if (name === 'calc') {
-                        call.mathOff = true;
-                    }
-                    return call;
+                    
+                    return new(tree.Call)(name, args, index, fileInfo);
                 },
 
                 //

lib/less/parser/parser.js

@@ -7,6 +7,7 @@ var Node = require("./node"),
 var Call = function (name, args, index, currentFileInfo) {
     this.name = name;
     this.args = args;
+    this.mathOn = name === 'calc' ? false : true;
     this._index = index;
     this._fileInfo = currentFileInfo;
 };
@@ -29,14 +30,15 @@ Call.prototype.accept = function (visitor) {
 // The function should receive the value, not the variable.
 //
 Call.prototype.eval = function (context) {
-    /* Used for calc() */
-    if (this.mathOff) {
-        context.mathOn = false;
-    }
+
+    /**
+     * Turn off math for calc(), and switch back on for evaluating nested functions
+     */
+    var currentMathContext = context.mathOn;
+    context.mathOn = this.mathOn;
     var args = this.args.map(function (a) { return a.eval(context); });
-    if (this.mathOff) {
-        context.mathOn = true;
-    }
+    context.mathOn = currentMathContext;
+
     var result, funcCaller = new FunctionCaller(this.name, context, this.getIndex(), this.fileInfo());
 
     if (funcCaller.isValid()) {

lib/less/tree/call.js

@@ -2,7 +2,7 @@ var less = {
     logLevel: 4, 
     errorReporting: "console", 
     javascriptEnabled: true,
-    strictMath: true
+    strictMath: false
 };
 
 // There originally run inside describe method. However, since they have not

test/browser/runner-browser-options.js

@@ -2,7 +2,6 @@ var less = {
     logLevel: 4,
     errorReporting: "console"
 };
-less.strictMath = true;
 less.functions = {
     add: function(a, b) {
         return new(less.tree.Dimension)(a.value + b.value);

test/browser/runner-main-options.js

@@ -1,3 +1,5 @@
 .no-math {
   width: calc(50% + (25vh - 20px));
+  foo: 3 calc(3 + 4) 11;
+  bar: calc(1 + 20%);
 }

test/css/calc.css

@@ -1,4 +1,7 @@
 .no-math {
   @var: 50vh/2;
   width: calc(50% + (@var - 20px));
+  foo: 1 + 2 calc(3 + 4) 5 + 6;
+  @floor: floor(1 + .1);
+  bar: calc(@floor + 20%);
 }