ion-schema-kotlin icon indicating copy to clipboard operation
ion-schema-kotlin copied to clipboard

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

Open pbcornell opened this issue 5 years ago • 0 comments

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 avatar Apr 24 '19 22:04 pbcornell