486
0%

Ran 03 Mar 2018 07:11PM UTC

Jobs 3

Files 8

Run time 33s

Badge

Embed ▾

pending completion

Build # 486

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	486.1	03 Mar 2018 07:11PM UTC	Travis Job 486.1
2	486.2	03 Mar 2018 07:11PM UTC	Travis Job 486.2
3	486.3	03 Mar 2018 07:11PM UTC	Travis Job 486.3

Source Files on build 486

Detailed source file information is not available for this build.

Khan / aphrodite / 486
0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 486

Khan / aphrodite / 486 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 486

Khan / aphrodite / 486
0%

README BADGES
x