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.

Sign up for GitHub

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

Jump to bottom

refactor range implementations in software.amazon.ionschema.internal.util #109

Open

pbcornell opened this issue Apr 24, 2019 · 0 comments

Milestone

pbcornell commented Apr 24, 2019

In particular, note that:

IntRange duplicates functionality provided by RangeInt, but offers some nice simplifications, as well as exposes the right interface for use by StateMachine
RangeFactory.rangeOf() is awkward; perhaps the codebase is better off without a factory here
the heavy use of delegation tends to be confusing; is there a way to simplify (or at least clarify) how this works?

pbcornell added this to the backlog milestone

popematt mentioned this issue

Adds ISL model #251

Merged

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