ion-java icon indicating copy to clipboard operation
ion-java copied to clipboard

Improve readability of IonReaderContinuableCoreBinary.nextValue() and transcodeNextTo()

Open tgregg opened this issue 1 year ago • 1 comments

These are lengthy methods with looping constructs and early returns. Figuring out a better factoring would improve readability and maintainability.

tgregg avatar Nov 27 '24 01:11 tgregg

Also apply this to IonReaderTextSystemX.transcodeNextTo().

Additionally, IonReaderContinuableCoreBinary.transcodeValueLiteral() and IonReaderTextSystemX.transcodeValueLiteral() are similar enough that we could consider making MacroAwareIonReader abstract and putting a generalized implementation in there.

tgregg avatar Dec 09 '24 20:12 tgregg