[
https://issues.apache.org/jira/browse/LUCENE-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13052660#comment-13052660
]
Robert Muir commented on LUCENE-3224:
-
also i think we want to assert all bounds checks in here, maybe have a
checkBounds(int limit) called only from assert that throws read past EOF.
this way we don't rely upon AIOOBE, we could be reading from slices and miss
bugs in tests.
bugs in ByteArrayDataInput
--
Key: LUCENE-3224
URL: https://issues.apache.org/jira/browse/LUCENE-3224
Project: Lucene - Java
Issue Type: Bug
Reporter: Robert Muir
ByteArrayDataInput has a byte[] ctor, but it doesn't actually work (some
things like readVint will work, others will fail due to asserts).
The problem is it doesnt set things like limit in the ctor... I think the
ctor should call reset()
Most code using this passes null to the ctor to initialize it, then uses
reset(), instead they could just call ByteArrayInput(BytesRef.EMPTY_BYTES) if
they want to do that.
finally, reset()'s limit looks like it should be offset + len
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org