Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unroll apply call further? #281

Open
danielcompton opened this issue Dec 21, 2016 · 0 comments
Open

Unroll apply call further? #281

danielcompton opened this issue Dec 21, 2016 · 0 comments

Comments

@danielcompton
Copy link
Contributor

danielcompton commented Dec 21, 2016

I was looking through some performance tracing and noticed in Chrome's devtools they now show cumulative times for lines of code. The .apply at the end of case statement got a lot of CPU time relative to other parts of the code (it probably was called a lot more though). There needs to be more validation that this is really a bottleneck worth addressing, but I thought I'd drop it here in the meantime as it seemed worth investigating further.

screenshot of google chrome 22-12-16 9-01-25 am

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

No branches or pull requests

2 participants