You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This works fine in Ion 1.0 because every value is length-prefixed, so the value marker's end index is always greater than or equal to its start index. However, the value markers for Ion 1.1's e-expressions and delimited containers will have end index = -1 until fully read. As currently implemented, this makes the value of OffsetSpan.getFinishOffset() garbage for these encodings. This should be fixed.
The text was updated successfully, but these errors were encountered:
The binary SpanProvider implementation sets the current span's end index using the value marker: https://github.com/amazon-ion/ion-java/blob/master/src/main/java/com/amazon/ion/impl/IonReaderContinuableTopLevelBinary.java#L269
This works fine in Ion 1.0 because every value is length-prefixed, so the value marker's end index is always greater than or equal to its start index. However, the value markers for Ion 1.1's e-expressions and delimited containers will have end index = -1 until fully read. As currently implemented, this makes the value of OffsetSpan.getFinishOffset() garbage for these encodings. This should be fixed.
The text was updated successfully, but these errors were encountered: