1
0%
master: 0%

Ran 03 Mar 2018 07:11PM UTC

Files 8

Run time 0s

Badge

Embed ▾

Committed 03 Mar 2018 07:09PM UTC coverage: 0.0%. Remained the same

Job # 486.1

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

Khan / aphrodite / 486 / 1
0%
master: 0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 486.1

Khan / aphrodite / 486 / 1 0% master: 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 486.1

Khan / aphrodite / 486 / 1
0%
master: 0%

README BADGES
x