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

Fix quadratic behaviour during emphasis processing #26

Merged
merged 5 commits into from
Jul 19, 2017
Merged

Conversation

kivikakk
Copy link
Owner

Fixes #24. Uses remstack.rs provided by @charliesome.

Not quite ready to merge yet; the correctness isn't 100%.

@kivikakk
Copy link
Owner Author

Next step: remove all messing around with indices entirely with opaque index/iterator type.

@kivikakk
Copy link
Owner Author

@zeantsoi I can recommend using a test repo that doesn't already have users with notifications subscribed 🐰

@kivikakk
Copy link
Owner Author

RemStack idea needs some development still, so for now I've rewritten this to use an Arena, keeping a lot closer to the C codebase.

@kivikakk kivikakk merged commit f59b709 into master Jul 19, 2017
@kivikakk kivikakk deleted the quadratic branch July 19, 2017 06:49
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.

1 participant