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

Merge pull request #5595 from filipesilva/fix-deopts 

fix: replace for..of to prevent v8 deoptimization
This commit is contained in:
Tobias Koppers 2017-08-31 11:22:12 +02:00 committed by GitHub
parent c8fd542579 2f30174d15
commit b75b20b416
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/SourceMapDevToolPlugin.js
View File

@ -96,7 +96,8 @@ class SourceMapDevToolPlugin {
return module || source;
});
for(const module of modules) {
for(let idx = 0; idx < modules.length; idx++) {
const module = modules[idx];
if(!moduleToSourceNameMapping.get(module)) {
moduleToSourceNameMapping.set(module, ModuleFilenameHelpers.createFilename(module, moduleFilenameTemplate, requestShortener));
}
@ -121,7 +122,8 @@ class SourceMapDevToolPlugin {
});
// find modules with conflicting source names
for(const module of allModules) {
for(let idx = 0; idx < allModules.length; idx++) {
const module = allModules[idx];
let sourceName = moduleToSourceNameMapping.get(module);
let hasName = conflictDetectionSet.has(sourceName);
if(!hasName) {

4
lib/dependencies/HarmonyModulesHelpers.js
View File

@ -34,8 +34,8 @@ class HarmonyModulesHelpers {
const desc = depInQuestion.describeHarmonyExport();
if(!desc.exportedName) return true;
let before = true;
for(const moduleDependency of module.dependencies) {
const dep = moduleDependency;
for(let idx = 0; idx < module.dependencies.length; idx++) {
const dep = module.dependencies[idx];
if(dep === depInQuestion) {
before = false;
continue;