484
0%
master: 0%

Ran 03 Mar 2018 07:04PM CUT

Jobs 3

Files 8

Run time 45s

Badge

Embed ▾

pending completion

Build # 484

Build Type

push

travis-ci

Committed by

lencioni

Commit Message

Remove separate iteration for summing length of style defs

I've been profiling and noticed that injectAndGetClassName is a very hot
path so I've been thinking about ways to make it more efficient. I
noticed that we were iterating over things twice unnecessarily, so I've
moved a reduce function inline with our original iteration. This
improves my naive benchmark from ~2.94s to ~2.77s.

While I was at it, I removed an unnecessary object creation here, to hlp
improve efficiency and reduce the memory footprint.

I think it might be possible to further improve perf here by converting
the recursive function into an interative one, which I'm going to dig
into next.

Run Details

0 of 0 relevant lines covered (NaN%)

0.0 hits per line

Jobs

ID	Job ID	Ran
1	484.1	03 Mar 2018 07:04PM CUT	Travis Job 484.1
2	484.2	03 Mar 2018 07:04PM CUT	Travis Job 484.2
3	484.3	03 Mar 2018 07:04PM CUT	Travis Job 484.3

Khan / aphrodite / 484
0%
master: 0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 484

Khan / aphrodite / 484 0% master: 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 484

Khan / aphrodite / 484
0%
master: 0%

README BADGES
x