Optimize "invisible" whitespace

Bug #1063984 reported by Lthere
6
This bug affects 1 person
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

Revision history for this message
Fenring (leunen-d) wrote :

Absolutely no objection.

Thanks for your great work Helder !

Changed in smil:
status: New → Confirmed
Changed in smil:
status: Confirmed → In Progress
summary: - "Invisible" optimize whitespace
+ Optimize "invisible" whitespace
Changed in smil:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.