Skip to content

Commit 0110035

Browse files
sokrasokra
authored
Merge pull request webpack#5497 from athomann/convert-binds
Convert .bind(this) to arrow functions
2 parents 143d272 + 15f4894 commit 0110035

5 files changed

Lines changed: 42 additions & 42 deletions

File tree

‎lib/Compilation.js‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -362,13 +362,13 @@ class Compilation extends Tapable {
362362
_addModuleChain(context, dependency, onModule, callback) {
363363
const start = this.profile && Date.now();
364364

365-
const errorAndCallback = this.bail ? function errorAndCallback(err) {
365+
const errorAndCallback = this.bail ? (err) => {
366366
callback(err);
367-
} : function errorAndCallback(err) {
367+
} : (err) => {
368368
err.dependencies = [dependency];
369369
this.errors.push(err);
370370
callback();
371-
}.bind(this);
371+
};
372372

373373
if(typeof dependency !== "object" || dependency === null || !dependency.constructor) {
374374
throw new Error("Parameter 'dependency' must be a Dependency");

‎lib/Compiler.js‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ class Compiler extends Tapable {
201201
});
202202
});
203203
},
204-
apply: function() {
204+
apply: () => {
205205
const args = arguments;
206206
if(!deprecationReported) {
207207
console.warn("webpack: Using compiler.parser is deprecated.\n" +
@@ -214,7 +214,7 @@ class Compiler extends Tapable {
214214
parser.apply.apply(parser, args);
215215
});
216216
});
217-
}.bind(this)
217+
}
218218
};
219219

220220
this.options = {};
@@ -313,13 +313,7 @@ class Compiler extends Tapable {
313313
emitAssets(compilation, callback) {
314314
let outputPath;
315315

316-
this.applyPluginsAsync("emit", compilation, err => {
317-
if(err) return callback(err);
318-
outputPath = compilation.getPath(this.outputPath);
319-
this.outputFileSystem.mkdirp(outputPath, emitFiles.bind(this));
320-
});
321-
322-
function emitFiles(err) {
316+
const emitFiles = (err) => {
323317
if(err) return callback(err);
324318

325319
require("async").forEach(Object.keys(compilation.assets), (file, callback) => {
@@ -330,12 +324,7 @@ class Compiler extends Tapable {
330324
targetFile = targetFile.substr(0, queryStringIdx);
331325
}
332326

333-
if(targetFile.match(/\/|\\/)) {
334-
const dir = path.dirname(targetFile);
335-
this.outputFileSystem.mkdirp(this.outputFileSystem.join(outputPath, dir), writeOut.bind(this));
336-
} else writeOut.call(this);
337-
338-
function writeOut(err) {
327+
const writeOut = (err) => {
339328
if(err) return callback(err);
340329
const targetPath = this.outputFileSystem.join(outputPath, targetFile);
341330
const source = compilation.assets[file];
@@ -352,14 +341,25 @@ class Compiler extends Tapable {
352341
source.existsAt = targetPath;
353342
source.emitted = true;
354343
this.outputFileSystem.writeFile(targetPath, content, callback);
355-
}
344+
};
345+
346+
if(targetFile.match(/\/|\\/)) {
347+
const dir = path.dirname(targetFile);
348+
this.outputFileSystem.mkdirp(this.outputFileSystem.join(outputPath, dir), writeOut);
349+
} else writeOut();
356350

357351
}, err => {
358352
if(err) return callback(err);
359353

360354
afterEmit.call(this);
361355
});
362-
}
356+
};
357+
358+
this.applyPluginsAsync("emit", compilation, err => {
359+
if(err) return callback(err);
360+
outputPath = compilation.getPath(this.outputPath);
361+
this.outputFileSystem.mkdirp(outputPath, emitFiles);
362+
});
363363

364364
function afterEmit() {
365365
this.applyPluginsAsyncSeries1("after-emit", compilation, err => {

‎lib/ContextModuleFactory.js‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -103,16 +103,16 @@ module.exports = class ContextModuleFactory extends Tapable {
103103
if(!regExp || !resource)
104104
return callback(null, []);
105105
(function addDirectory(directory, callback) {
106-
fs.readdir(directory, function(err, files) {
106+
fs.readdir(directory, (err, files) => {
107107
if(err) return callback(err);
108108
if(!files || files.length === 0) return callback(null, []);
109109
asyncLib.map(files.filter(function(p) {
110110
return p.indexOf(".") !== 0;
111-
}), function(seqment, callback) {
111+
}), (seqment, callback) => {
112112

113113
const subResource = path.join(directory, seqment);
114114

115-
fs.stat(subResource, function(err, stat) {
115+
fs.stat(subResource, (err, stat) => {
116116
if(err) return callback(err);
117117

118118
if(stat.isDirectory()) {
@@ -141,9 +141,9 @@ module.exports = class ContextModuleFactory extends Tapable {
141141

142142
} else callback();
143143

144-
}.bind(this));
144+
});
145145

146-
}.bind(this), (err, result) => {
146+
}, (err, result) => {
147147
if(err) return callback(err);
148148

149149
if(!result) return callback(null, []);
@@ -154,7 +154,7 @@ module.exports = class ContextModuleFactory extends Tapable {
154154
return a.concat(i);
155155
}, []));
156156
});
157-
}.bind(this));
157+
});
158158
}.call(this, resource, callback));
159159
}
160160
};

‎lib/Parser.js‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -634,14 +634,14 @@ class Parser extends Tapable {
634634
statement.params.forEach(param => {
635635
this.walkPattern(param);
636636
});
637-
this.inScope(statement.params, function() {
637+
this.inScope(statement.params, () => {
638638
if(statement.body.type === "BlockStatement") {
639639
this.prewalkStatement(statement.body);
640640
this.walkStatement(statement.body);
641641
} else {
642642
this.walkExpression(statement.body);
643643
}
644-
}.bind(this));
644+
});
645645
}
646646

647647
prewalkImportDeclaration(statement) {
@@ -784,10 +784,10 @@ class Parser extends Tapable {
784784
}
785785

786786
walkCatchClause(catchClause) {
787-
this.inScope([catchClause.param], function() {
787+
this.inScope([catchClause.param], () => {
788788
this.prewalkStatement(catchClause.body);
789789
this.walkStatement(catchClause.body);
790-
}.bind(this));
790+
});
791791
}
792792

793793
prewalkVariableDeclarators(declarators) {
@@ -916,28 +916,28 @@ class Parser extends Tapable {
916916
expression.params.forEach(param => {
917917
this.walkPattern(param);
918918
});
919-
this.inScope(expression.params, function() {
919+
this.inScope(expression.params, () => {
920920
if(expression.body.type === "BlockStatement") {
921921
this.prewalkStatement(expression.body);
922922
this.walkStatement(expression.body);
923923
} else {
924924
this.walkExpression(expression.body);
925925
}
926-
}.bind(this));
926+
});
927927
}
928928

929929
walkArrowFunctionExpression(expression) {
930930
expression.params.forEach(param => {
931931
this.walkPattern(param);
932932
});
933-
this.inScope(expression.params, function() {
933+
this.inScope(expression.params, () => {
934934
if(expression.body.type === "BlockStatement") {
935935
this.prewalkStatement(expression.body);
936936
this.walkStatement(expression.body);
937937
} else {
938938
this.walkExpression(expression.body);
939939
}
940-
}.bind(this));
940+
});
941941
}
942942

943943
walkSequenceExpression(expression) {
@@ -1059,7 +1059,7 @@ class Parser extends Tapable {
10591059
const args = options.map(renameArgOrThis, this);
10601060
this.inScope(params.filter(function(identifier, idx) {
10611061
return !args[idx];
1062-
}), function() {
1062+
}), () => {
10631063
if(renameThis) {
10641064
this.scope.renames.$this = renameThis;
10651065
}
@@ -1074,7 +1074,7 @@ class Parser extends Tapable {
10741074
this.walkStatement(functionExpression.body);
10751075
} else
10761076
this.walkExpression(functionExpression.body);
1077-
}.bind(this));
1077+
});
10781078
}
10791079
if(expression.callee.type === "MemberExpression" &&
10801080
expression.callee.object.type === "FunctionExpression" &&

‎lib/UmdMainTemplatePlugin.js‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ class UmdMainTemplatePlugin {
4141

4242
apply(compilation) {
4343
const mainTemplate = compilation.mainTemplate;
44-
compilation.templatesPlugin("render-with-entry", function(source, chunk, hash) {
44+
compilation.templatesPlugin("render-with-entry", (source, chunk, hash) => {
4545
let externals = chunk.getModules().filter(m => m.external);
4646
const optionalExternals = [];
4747
let requiredExternals = [];
@@ -172,19 +172,19 @@ class UmdMainTemplatePlugin {
172172
" }\n"
173173
) +
174174
"})(this, function(" + externalsArguments(externals) + ") {\nreturn ", "webpack/universalModuleDefinition"), source, ";\n})");
175-
}.bind(this));
176-
mainTemplate.plugin("global-hash-paths", function(paths) {
175+
});
176+
mainTemplate.plugin("global-hash-paths", (paths) => {
177177
if(this.names.root) paths = paths.concat(this.names.root);
178178
if(this.names.amd) paths = paths.concat(this.names.amd);
179179
if(this.names.commonjs) paths = paths.concat(this.names.commonjs);
180180
return paths;
181-
}.bind(this));
182-
mainTemplate.plugin("hash", function(hash) {
181+
});
182+
mainTemplate.plugin("hash", (hash) => {
183183
hash.update("umd");
184184
hash.update(`${this.names.root}`);
185185
hash.update(`${this.names.amd}`);
186186
hash.update(`${this.names.commonjs}`);
187-
}.bind(this));
187+
});
188188
}
189189
}
190190

0 commit comments

Comments
 (0)