Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improving performance and GC issues #125

Merged
merged 1 commit into from
Mar 31, 2016
Merged

Improving performance and GC issues #125

merged 1 commit into from
Mar 31, 2016

Conversation

qfox
Copy link
Contributor

@qfox qfox commented Mar 31, 2016

We catching a lot of problems on using the current version of tmpl-specs because 6 templators on several hundreds of samples makes nodejs going oom.
Also it improves performance. So MERGE IT MERGE IT

var bemjson = references['${ it }']${ subreference }.bemjson,
expected = references['${ it }']${ subreference }.html;<%

if (!engine.async) { %>
// sync mode
var actual = engines['${ engine.name }']${ subreference }.apply(bemjson);
var actual = engine.apply(bemjson);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So true, eagle eye!

Thank you.

Rework a little where and how clearRequire-require combo works.
Just it improved performance a lot.

Also stopped usign this combo for references.
@blond blond merged commit 30872bb into master Mar 31, 2016
@blond blond deleted the zxqfox.perf-rerequire branch March 31, 2016 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants