>
> Christoph Kiehl wrote:
> I made some changes to the ChildAxisScorer while working on JCR-1041.
> Unfortunately I don't have the time right now to check if I
> introduced this bug.
> I'll try to have a look later this day. Feel free to check
> the affected revisions.
When reverting to revision before these changes the problem did not occur. The problem appears
to be, that in
ChildAxisQuery.ChildAxisScorer.skiptTo(int) the skipTo(target) can return -1 (leading to problem
in reader.document(i) where i = -1 ). adding
if(nextDoc == -1) {
return false;
}
after nextDoc in the code snippet below solves the problem. But, I am not sure wether it should
be avoided in the skipTo in the first place?
public boolean skipTo(int target) throws IOException {
calculateChildren();
nextDoc = hits.skipTo(target);
while (!indexIsValid(nextDoc))
next();
return nextDoc > -1;
}
WDYT? Shall I re-open the JCR-1041?
Regards Ard
>
> Cheers,
> Christoph
>
>
|