Filed the bug, https://issues.apache.org/jira/browse/LUCENE-511 .
Also added the simple fix, it seems to work, and I think the rest of
the method is ok.
Shay
On 2 Mar 2006, at 19:19, Doug Cutting wrote:
Shay Banon wrote:
...
} else {
// is data larger then buffer?
if (length > BUFFER_SIZE) {
// we flush the buffer
if (bufferPosition > 0)
flush();
// and write data at once
flushBuffer(b, length);
} else {
...
the bufferStart is not incremented after the flushBuffer method
is called. So if someone calls getFilePointer just afterwards, it
will give the wrong result (hit it with the compound format). A
simple fix would be to add bufferStart += length; just after
flushBuffer.
Can you please file a bug for this and attach a bug to it with a
unit test that illustrates the problem? This looks like something
that could warrant a 1.9.1 release, so we must proceed carefully.
Thanks,
Doug
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
