vaadin / sass-compiler

A Java Sass compiler implementation
53 stars 26 forks source link

SassCompiler: various micro optimizations #271

Open vaadin-bot opened 10 years ago

vaadin-bot commented 10 years ago

Originally by CodingFabian


Over the weekend I spend some time experimenting with various profilers to look for code-change potential in sass-compiler. After #14611 the runtime is not really dominated by any single code anymore. It is mostly that the sheer number of invocations and the generated parser code is having its impact. I did implement a few changed, for which I will add a review now, which I propose to be made. Their individual impact is pretty low, but together you can see differences in single-shot or benchmark invocations of valo theme. All changes for themselves are straightforward. If you dislike any specific let me know and i will remove them.


Imported from https://dev.vaadin.com/ issue #14623

vaadin-bot commented 10 years ago

Originally by CodingFabian


review added here: https://dev.vaadin.com/review/#/c/4637/