root
/
webpack
mirror of https://github.com/webpack/webpack.git
1
0
Fork 0
Code Issues Actions Packages Projects Releases Wiki Activity

Use Set for commonModules. Use getModules instead of modules

This commit is contained in:
Tobias Koppers 2017-09-22 15:44:16 +02:00
parent 36544088b7
commit 845c5f5031
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/optimize/AggressiveMergingPlugin.js
View File

@ -77,14 +77,14 @@ class AggressiveMergingPlugin {
if(pair.improvement < minSizeReduce) return;
if(options.moveToParents) {
const commonModules = pair.b.modules.filter((m) => {
return pair.a.modules.indexOf(m) >= 0;
const commonModules = new Set(pair.b.getModules().filter((m) => {
return pair.a.hasModule(m);
}));
const aOnlyModules = pair.b.getModules().filter((m) => {
return !commonModules.has(m);
});
const aOnlyModules = pair.b.modules.filter((m) => {
return commonModules.indexOf(m) < 0;
});
const bOnlyModules = pair.a.modules.filter((m) => {
return commonModules.indexOf(m) < 0;
const bOnlyModules = pair.a.getModules().filter((m) => {
return !commonModules.has(m);
});
aOnlyModules.forEach((m) => {
pair.b.removeModule(m);