Faster onEachEvent event (including namespaces) #18

Merged
merged 1 commit into from Apr 7, 2016

Projects

None yet

3 participants

@Ke-
Contributor
Ke- commented Apr 6, 2016

Roughly 30% faster with simple events:

new-observable#simple event x 1,801,581 ops/sec ±1.03% (87 runs sampled)
new-observable#namespaced event x 1,697,375 ops/sec ±4.13% (85 runs sampled)
new-observable#multiple events x 259,675 ops/sec ±1.62% (82 runs sampled)

old-observable#simple event x 1,385,113 ops/sec ±1.00% (88 runs sampled)
old-observable#multiple events x 238,766 ops/sec ±3.05% (81 runs sampled)
@GianlucaGuarini
Member

this looks perfect to me thank you so much! the version 2.4.0 will come along

@GianlucaGuarini GianlucaGuarini merged commit a235a44 into riot:master Apr 7, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rma4ok
rma4ok commented May 9, 2016

@Ke- You are awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment