Parenthesize mixed expressions of || and && (fix #2949)

ExtendScript apparently gives equal precedence to the || and &&
operators, so when the || appears before the &&, we need to anchor the
order of evaluation with an extra pair of parentheses.

Author: jgonggrijp

modules/_setup.js

@@ -4,8 +4,8 @@ export var VERSION = '1.13.2';
 // Establish the root object, `window` (`self`) in the browser, `global`
 // on the server, or `this` in some virtual machines. We use `self`
 // instead of `window` for `WebWorker` support.
-export var root = typeof self == 'object' && self.self === self && self ||
-          typeof global == 'object' && global.global === global && global ||
+export var root = (typeof self == 'object' && self.self === self && self) ||
+          (typeof global == 'object' && global.global === global && global) ||
           Function('return this')() ||
           {};
 

modules/isObject.js

@@ -1,5 +1,5 @@
 // Is a given variable an object?
 export default function isObject(obj) {
   var type = typeof obj;
-  return type === 'function' || type === 'object' && !!obj;
+  return type === 'function' || (type === 'object' && !!obj);
 }

modules/max.js

@@ -7,7 +7,7 @@ import each from './each.js';
 export default function max(obj, iteratee, context) {
   var result = -Infinity, lastComputed = -Infinity,
       value, computed;
-  if (iteratee == null || typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null) {
+  if (iteratee == null || (typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null)) {
     obj = isArrayLike(obj) ? obj : values(obj);
     for (var i = 0, length = obj.length; i < length; i++) {
       value = obj[i];
@@ -19,7 +19,7 @@ export default function max(obj, iteratee, context) {
     iteratee = cb(iteratee, context);
     each(obj, function(v, index, list) {
       computed = iteratee(v, index, list);
-      if (computed > lastComputed || computed === -Infinity && result === -Infinity) {
+      if (computed > lastComputed || (computed === -Infinity && result === -Infinity)) {
         result = v;
         lastComputed = computed;
       }

modules/min.js

@@ -7,7 +7,7 @@ import each from './each.js';
 export default function min(obj, iteratee, context) {
   var result = Infinity, lastComputed = Infinity,
       value, computed;
-  if (iteratee == null || typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null) {
+  if (iteratee == null || (typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null)) {
     obj = isArrayLike(obj) ? obj : values(obj);
     for (var i = 0, length = obj.length; i < length; i++) {
       value = obj[i];
@@ -19,7 +19,7 @@ export default function min(obj, iteratee, context) {
     iteratee = cb(iteratee, context);
     each(obj, function(v, index, list) {
       computed = iteratee(v, index, list);
-      if (computed < lastComputed || computed === Infinity && result === Infinity) {
+      if (computed < lastComputed || (computed === Infinity && result === Infinity)) {
         result = v;
         lastComputed = computed;
       }

underscore-esm.js

@@ -11,8 +11,8 @@ var VERSION = '1.13.2';
 // Establish the root object, `window` (`self`) in the browser, `global`
 // on the server, or `this` in some virtual machines. We use `self`
 // instead of `window` for `WebWorker` support.
-var root = typeof self == 'object' && self.self === self && self ||
-          typeof global == 'object' && global.global === global && global ||
+var root = (typeof self == 'object' && self.self === self && self) ||
+          (typeof global == 'object' && global.global === global && global) ||
           Function('return this')() ||
           {};
 
@@ -80,7 +80,7 @@ function restArguments(func, startIndex) {
 // Is a given variable an object?
 function isObject(obj) {
   var type = typeof obj;
-  return type === 'function' || type === 'object' && !!obj;
+  return type === 'function' || (type === 'object' && !!obj);
 }
 
 // Is a given value equal to null?
@@ -1460,7 +1460,7 @@ function where(obj, attrs) {
 function max(obj, iteratee, context) {
   var result = -Infinity, lastComputed = -Infinity,
       value, computed;
-  if (iteratee == null || typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null) {
+  if (iteratee == null || (typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null)) {
     obj = isArrayLike(obj) ? obj : values(obj);
     for (var i = 0, length = obj.length; i < length; i++) {
       value = obj[i];
@@ -1472,7 +1472,7 @@ function max(obj, iteratee, context) {
     iteratee = cb(iteratee, context);
     each(obj, function(v, index, list) {
       computed = iteratee(v, index, list);
-      if (computed > lastComputed || computed === -Infinity && result === -Infinity) {
+      if (computed > lastComputed || (computed === -Infinity && result === -Infinity)) {
         result = v;
         lastComputed = computed;
       }
@@ -1485,7 +1485,7 @@ function max(obj, iteratee, context) {
 function min(obj, iteratee, context) {
   var result = Infinity, lastComputed = Infinity,
       value, computed;
-  if (iteratee == null || typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null) {
+  if (iteratee == null || (typeof iteratee == 'number' && typeof obj[0] != 'object' && obj != null)) {
     obj = isArrayLike(obj) ? obj : values(obj);
     for (var i = 0, length = obj.length; i < length; i++) {
       value = obj[i];
@@ -1497,7 +1497,7 @@ function min(obj, iteratee, context) {
     iteratee = cb(iteratee, context);
     each(obj, function(v, index, list) {
       computed = iteratee(v, index, list);
-      if (computed < lastComputed || computed === Infinity && result === Infinity) {
+      if (computed < lastComputed || (computed === Infinity && result === Infinity)) {
         result = v;
         lastComputed = computed;
       }