464
0%
master: 0%

Ran 16 Feb 2018 07:35PM UTC

Jobs 1

Files 8

Run time 1s

Badge

Embed ▾

pending completion

Build # 464

Build Type

push

travis-ci

Committed by

lencioni

Commit Message

Replace object spread with Set in generateCSSRuleset

I was profiling Aphrodite and noticed that the self time in
generateCSSRuleset is one of the most expensive bits. Digging in a bit,
it looked like most of this was coming from the object spread here,
which we use do copy the original object so we can reference its keys
with fast lookups. By swapping this out for a Set we greatly reduce the
self time of this function from ~7% of total runtime to ~3%.

I've implemented this in a way that expects Set to be available in the
browser or polyfilled, which I think is a reasonable explanation. I've
documented this in the readme. If we move forward with this, we could
also make Map a requirement and simplify a couple of codepaths if we
want.

Run Details

0 of 0 relevant lines covered (NaN%)

0.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	464.1	16 Feb 2018 07:35PM UTC	0	0.0	Travis Job 464.1

Source Files on build 464

Detailed source file information is not available for this build.

Khan / aphrodite / 464
0%
master: 0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 464

Khan / aphrodite / 464 0% master: 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 464

Khan / aphrodite / 464
0%
master: 0%

README BADGES
x