Optimize "invisible" whitespace
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
FakeSmile |
Fix Committed
|
Wishlist
|
Lthere |
Bug Description
Simple replacements have been made in order to understand the benefits. The idea is to optimize whitespace while retaining all code readability.
Numbers assume that Sizzle removal (bug #1038928) has already took place, as it is a relevant piece of the library which was already optimized ("aggressively": newlines and such were removed, variable names optimized, etc.).
So here are the results:
1. Rework two spaces for tab character: 7% improvement (!)
2. EOL changed to Unix: 3% improvement
We see here that the two together lead to a library size improvement of about 10% without any special readability loss, so it's quite low-hanging fruit.
I'm hoping to get my hands on this soon so comments are welcome (that is, are there any good arguments against this optimization? positive feedback also appreciated).
Related branches
Changed in smil: | |
status: | Confirmed → In Progress |
summary: |
- "Invisible" optimize whitespace + Optimize "invisible" whitespace |
Changed in smil: | |
status: | In Progress → Fix Committed |
Absolutely no objection.
Thanks for your great work Helder !