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 #197 #198

Closed
wants to merge 4 commits into from
Closed

Fix #197 #198

wants to merge 4 commits into from

Conversation

bgamari
Copy link
Contributor

@bgamari bgamari commented Sep 8, 2017

This fixes a variety of size hint bugs in text's fusion framework. These could manifest in a variety of issues similar to #197.

Otherwise we may miss bugs revealed by rewrite rules.
The length of the input represents the upper bound of filter's result size. This
was not reflected in its implementation.
@bgamari
Copy link
Contributor Author

bgamari commented Sep 29, 2017

Thanks to reminders from @hvr and @nomeata I'm going to pick this up again.

@bgamari bgamari changed the title [WIP] Fix #197 Fix #197 Oct 9, 2017
@bgamari
Copy link
Contributor Author

bgamari commented Oct 9, 2017

Arg, yet again I regret opening a PR from my master branch.

@bgamari bgamari closed this Oct 9, 2017
@bgamari
Copy link
Contributor Author

bgamari commented Oct 9, 2017

This has been superceded by #200.

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.

None yet

2 participants