improve performance of ModuleConcatenationPlugin for loop

brentwilton · brentwilton · commit 9f37b499bfd4 · 2018-02-28T11:46:12.000+13:00
diff --git a/lib/optimize/ModuleConcatenationPlugin.js b/lib/optimize/ModuleConcatenationPlugin.js
@@ -281,9 +281,12 @@ class ModuleConcatenationPlugin {
 							for (const reason of newModule.reasons) {
 								reason.dependency.module = newModule;
 							}
-							for (const dep of newModule.dependencies) {
+							for (let i = 0; i < newModule.dependencies.length; i++) {
+								let dep = newModule.dependencies[i];
 								if (dep.module) {
-									for (const reason of dep.module.reasons) {
+									let reasons = dep.module.reasons;
+									for (let j = 0; j < reasons.length; j++) {
+										let reason = reasons[j];
 										if (reason.dependency === dep) reason.module = newModule;
 									}
 								}

Original file line number	Diff line number	Diff line change
`@@ -281,9 +281,12 @@ class ModuleConcatenationPlugin {`
`281`	`281`	`for (const reason of newModule.reasons) {`
`282`	`282`	`reason.dependency.module = newModule;`
`283`	`283`	`}`
`284`		`- for (const dep of newModule.dependencies) {`
	`284`	`+ for (let i = 0; i < newModule.dependencies.length; i++) {`
	`285`	`+ let dep = newModule.dependencies[i];`
`285`	`286`	`if (dep.module) {`
`286`		`- for (const reason of dep.module.reasons) {`
	`287`	`+ let reasons = dep.module.reasons;`
	`288`	`+ for (let j = 0; j < reasons.length; j++) {`
	`289`	`+ let reason = reasons[j];`
`287`	`290`	`if (reason.dependency === dep) reason.module = newModule;`
`288`	`291`	`}`
`289`	`292`	`}`