-
Notifications
You must be signed in to change notification settings - Fork 12
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
Potential memory leak? #25
Labels
bug
Something isn't working
Comments
Hey. What's the status on this? Did the new runtime help with this problem? |
Good point, yes it did. With the new 0.18 runtime the performance is improved and also I’ve been able to add some more complex regex tests to the benchmark 12c8e55 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've been expanding the scope of the benchmark tests, most recently adding a more complex regex (https://github.com/ColinEberhardt/assemblyscript-regex/pull/24/files#diff-42e96713f3210182cc2ac65586ac4e2025e9c8134b31f88b2da3c2f69be5f452R65) to support evaluation of algorithm changes (e.g. NFA => DFA).
However, adding this more complex regex has caused an issue which looks like it could indicate a memory leak:
see the PR: #24
The text was updated successfully, but these errors were encountered: