AtHeartEngineer/ejs
1
0
Fork 0
mirror of https://github.com/CryptKeeperZK/ejs.git synced 2026-01-07 22:53:52 -05:00
Code Issues Packages Projects Releases Wiki Activity

Use string concatenation to build the output string

Browse Source 
Switching from pushing to an array and using .join() to using simple string concatenation yields performance gains up to 260%.
This commit is contained in:
Nathan Woltman
2019-10-30 22:17:37 -04:00
parent da51ca331a
commit 4434314d53
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
benchmark/bench-ejs.js
View File

@@ -1,8 +1,9 @@
'use strict';
var ejs = require('..');
var path = require('path');
ejs.fileLoader = function(n) { return files[n.replace(/^\//, '').replace(/\.ejs$/, '')]; };
ejs.fileLoader = function(n) { return files[path.basename(n, '.ejs')]; };
var loops = 10000;
var runs = 9; // min 4 for median

6
lib/ejs.js
View File

@@ -572,7 +572,9 @@ Template.prototype = {
if (!this.source) {
this.generateSource();
prepended += ' var __output = [], __append = __output.push.bind(__output);' + '\n';
prepended +=
' var __output = "";\n' +
' function __append(s) { if (s !== undefined && s !== null) __output += s }\n';
if (opts.outputFunctionName) {
prepended += ' var ' + opts.outputFunctionName + ' = __append;' + '\n';
}
@@ -591,7 +593,7 @@ Template.prototype = {
prepended += ' with (' + opts.localsName + ' || {}) {' + '\n';
appended += ' }' + '\n';
}
appended += ' return __output.join("");' + '\n';
appended += ' return __output;' + '\n';
this.source = prepended + this.source + appended;
}