Skip to content

speed increase attempt #629

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

Merged
merged 13 commits into from
Sep 11, 2016
Merged

speed increase attempt #629

merged 13 commits into from
Sep 11, 2016

Conversation

bounceme
Copy link
Collaborator

No description provided.

@bounceme
Copy link
Collaborator Author

Testing with https://github.com/sanctuary-js/sanctuary-def/blob/master/index.js

before: ~11s
after: 3.86s

@bounceme bounceme merged commit 0b6aad4 into develop Sep 11, 2016
@bounceme bounceme deleted the bounceme-patch-1 branch September 11, 2016 20:43
@amadeus
Copy link
Collaborator

amadeus commented Sep 12, 2016

This is pretty huge if it doesn't introduce any side effects!

@bounceme
Copy link
Collaborator Author

Definitely happy about this! also it is 100% accurate ( or could be if there is something i've missed ). It is only used for

let num = s:GetPair(pattern[0],pattern[1],'bW','s:skip_func(s:looksyn)',2000)
else
let num = s:GetPair('[({[]','[])}]','bW','s:skip_func(s:looksyn)',2000)
since that is the major bottleneck

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

Successfully merging this pull request may close these issues.

2 participants