build(broccoli): don't set stored diffResult to null on use

Doing this would break trees which are used multiple times.
angular · Jun 29, 2015 · 73b3ed1 · 73b3ed1
1 parent a93ec73
commit 73b3ed1
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/tools/broccoli/diffing-broccoli-plugin.ts b/tools/broccoli/diffing-broccoli-plugin.ts
@@ -75,12 +75,9 @@ class DiffingPluginWrapper implements BroccoliTree {
       // inputTree's rebuild() method, and can be used without being checked.
       // Set `this.diffResult` to null and return the previously stored value.
       let diffResult = tree.diffResult;
-      tree.diffResult = null;
-      if (!diffResult) {
-        let differ = index === false ? this.treeDiffer : this.treeDiffers[index];
-        diffResult = differ.diffTree();
-      }
-      return diffResult;
+      if (diffResult) return diffResult;
+      let differ = index === false ? this.treeDiffer : this.treeDiffers[index];
+      return differ.diffTree();
     };
 
     if (this.inputTrees) {