removed utils/clone, added .cloneDeep() where clone() was used (google#235)

Author: abeledovictor

lib/equation/Equation.js

@@ -1,6 +1,5 @@
 const math = require('mathjs');
 
-const clone = require('../util/clone');
 const printNode = require('../util/print');
 
 // This represents an equation, made up of the leftNode (LHS), the
@@ -31,8 +30,8 @@ class Equation {
   }
 
   clone() {
-    const newLeft = clone(this.leftNode);
-    const newRight = clone(this.rightNode);
+    const newLeft = this.leftNode.cloneDeep();
+    const newRight = this.rightNode.cloneDeep();
     return new Equation(newLeft, newRight, this.comparator);
   }
 }

lib/node/Status.js

@@ -1,5 +1,3 @@
-const clone = require('../util/clone');
-
 const ChangeTypes = require('../ChangeTypes');
 const Type = require('./Type');
 
@@ -29,7 +27,7 @@ class Status {
 }
 
 Status.resetChangeGroups = function(node) {
-  node = clone(node);
+  node = node.cloneDeep();
   node.filter(node => node.changeGroup).forEach(change => {
     delete change.changeGroup;
   });
@@ -60,8 +58,8 @@ Status.nodeChanged = function(
 // updated to have the newNode/oldNode metadata (changeGroups)
 // e.g. (2 + 2) + x --> 4 + x has to update the left argument
 Status.childChanged = function(node, childStatus, childArgIndex=null) {
-  const oldNode = clone(node);
-  const newNode = clone(node);
+  const oldNode = node.cloneDeep();
+  const newNode = node.cloneDeep();
   let substeps = childStatus.substeps;
 
   if (!childStatus.oldNode) {
@@ -80,8 +78,8 @@ Status.childChanged = function(node, childStatus, childArgIndex=null) {
     oldNode.content = childStatus.oldNode;
     newNode.content = childStatus.newNode;
     substeps = updateSubsteps(substeps, (step) => {
-      const oldNode = clone(node);
-      const newNode = clone(node);
+      const oldNode = node.cloneDeep();
+      const newNode = node.cloneDeep();
       oldNode.content = step.oldNode;
       newNode.content = step.newNode;
       step.oldNode = oldNode;
@@ -94,8 +92,8 @@ Status.childChanged = function(node, childStatus, childArgIndex=null) {
     oldNode.args[childArgIndex] = childStatus.oldNode;
     newNode.args[childArgIndex] = childStatus.newNode;
     substeps = updateSubsteps(substeps, (step) => {
-      const oldNode = clone(node);
-      const newNode = clone(node);
+      const oldNode = node.cloneDeep();
+      const newNode = node.cloneDeep();
       oldNode.args[childArgIndex] = step.oldNode;
       newNode.args[childArgIndex] = step.newNode;
       step.oldNode = oldNode;
@@ -107,8 +105,8 @@ Status.childChanged = function(node, childStatus, childArgIndex=null) {
     oldNode.args[0] = childStatus.oldNode;
     newNode.args[0] = childStatus.newNode;
     substeps = updateSubsteps(substeps, (step) => {
-      const oldNode = clone(node);
-      const newNode = clone(node);
+      const oldNode = node.cloneDeep();
+      const newNode = node.cloneDeep();
       oldNode.args[0] = step.oldNode;
       newNode.args[0] = step.newNode;
       step.oldNode = oldNode;

lib/simplifyExpression/basicsSearch/convertMixedNumberToImproperFraction.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
 
@@ -12,7 +10,7 @@ function convertMixedNumberToImproperFraction(node) {
   }
 
   const substeps = [];
-  let newNode = clone(node);
+  let newNode = node.cloneDeep();
 
   // e.g. 1 2/3
   const wholeNumber = Node.MixedNumber.getWholeNumberValue(node); // 1

lib/simplifyExpression/basicsSearch/rearrangeCoefficient.js

@@ -1,5 +1,4 @@
 const checks = require('../../checks');
-const clone = require('../../util/clone');
 
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
@@ -12,7 +11,7 @@ function rearrangeCoefficient(node) {
     return Node.Status.noChange(node);
   }
 
-  let newNode = clone(node);
+  let newNode = node.cloneDeep();
 
   const polyNode = new Node.PolynomialTerm(newNode.args[0]);
   const constNode = newNode.args[1];

lib/simplifyExpression/basicsSearch/removeAdditionOfZero.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
 
@@ -12,7 +10,7 @@ function removeAdditionOfZero(node) {
   const zeroIndex = node.args.findIndex(arg => {
     return Node.Type.isConstant(arg) && arg.value === '0';
   });
-  let newNode = clone(node);
+  let newNode = node.cloneDeep();
   if (zeroIndex >= 0) {
     // remove the 0 node
     newNode.args.splice(zeroIndex, 1);

lib/simplifyExpression/basicsSearch/removeDivisionByOne.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Negative = require('../../Negative');
 const Node = require('../../node');
@@ -14,7 +12,9 @@ function removeDivisionByOne(node) {
   if (!Node.Type.isConstant(denominator)) {
     return Node.Status.noChange(node);
   }
-  let numerator = clone(node.args[0]);
+  // It's taken 40ms on average to pass distribution test,
+  // TODO: see if we should keep using utils/clone here
+  let numerator = node.args[0].cloneDeep();
 
   // if denominator is -1, we make the numerator negative
   if (parseFloat(denominator.value) === -1) {

lib/simplifyExpression/basicsSearch/removeExponentBaseOne.js

@@ -1,5 +1,4 @@
 const checks = require('../../checks');
-const clone = require('../../util/clone');
 
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
@@ -11,7 +10,7 @@ function removeExponentBaseOne(node) {
       checks.resolvesToConstant(node.args[1]) && // a power not a symbol and
       Node.Type.isConstant(node.args[0]) &&      // a constant base
       node.args[0].value === '1') {              // of value 1
-    const newNode = clone(node.args[0]);
+    const newNode = node.args[0].cloneDeep();
     return Node.Status.nodeChanged(
       ChangeTypes.REMOVE_EXPONENT_BASE_ONE, node, newNode);
   }

lib/simplifyExpression/basicsSearch/removeExponentByOne.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
 
@@ -9,7 +7,7 @@ function removeExponentByOne(node) {
   if (node.op === '^' &&                   // exponent of anything
       Node.Type.isConstant(node.args[1]) && // to a constant
       node.args[1].value === '1') {        // of value 1
-    const newNode = clone(node.args[0]);
+    const newNode = node.args[0].cloneDeep();
     return Node.Status.nodeChanged(
       ChangeTypes.REMOVE_EXPONENT_BY_ONE, node, newNode);
   }

lib/simplifyExpression/basicsSearch/removeMultiplicationByNegativeOne.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Negative = require('../../Negative');
 const Node = require('../../node');
@@ -36,10 +34,10 @@ function removeMultiplicationByNegativeOne(node) {
     return Node.Status.noChange(node);
   }
 
-  let newNode = clone(node);
+  let newNode = node.cloneDeep();
 
   // Get rid of the -1
-  nodeToCombine = Negative.negate(clone(nodeToCombine));
+  nodeToCombine = Negative.negate(nodeToCombine.cloneDeep());
 
   // replace the node next to -1 and remove -1
   newNode.args[nodeToCombineIndex] = nodeToCombine;

lib/simplifyExpression/basicsSearch/removeMultiplicationByOne.js

@@ -1,5 +1,3 @@
-const clone = require('../../util/clone');
-
 const ChangeTypes = require('../../ChangeTypes');
 const Node = require('../../node');
 
@@ -13,7 +11,7 @@ function removeMultiplicationByOne(node) {
     return Node.Type.isConstant(arg) && arg.value === '1';
   });
   if (oneIndex >= 0) {
-    let newNode = clone(node);
+    let newNode = node.cloneDeep();
     // remove the 1 node
     newNode.args.splice(oneIndex, 1);
     // if there's only one operand left, there's nothing left to multiply it