Re: Bug in BooleanScorer2

2006-06-21 Thread DM Smith
On 6/21/06, Chris Hostetter <[EMAIL PROTECTED]> wrote: : > I haven't looed at the code so i'm not sure ... but my first thought would : > be to try both this.doc() and BooleanScorer2.this.doc(), and see which one : > passes all the unit tests. Of course, if both of them pass the unit : > tests

Re: Bug in BooleanScorer2

2006-06-21 Thread Yonik Seeley
On 6/21/06, DM Smith <[EMAIL PROTECTED]> wrote: On 6/21/06, Chris Hostetter <[EMAIL PROTECTED]> wrote: > > > : Not sure what the solution should be: > > I haven't looed at the code so i'm not sure ... but my first thought would > be to try both this.doc() and BooleanScorer2.this.doc(), and see wh

Re: Bug in BooleanScorer2

2006-06-21 Thread Chris Hostetter
: > I haven't looed at the code so i'm not sure ... but my first thought would : > be to try both this.doc() and BooleanScorer2.this.doc(), and see which one : > passes all the unit tests. Of course, if both of them pass the unit : > tests, then I'd be *really* worried. : Start worrying! : this.

Re: Bug in BooleanScorer2

2006-06-21 Thread DM Smith
On 6/21/06, Chris Hostetter <[EMAIL PROTECTED]> wrote: : Not sure what the solution should be: I haven't looed at the code so i'm not sure ... but my first thought would be to try both this.doc() and BooleanScorer2.this.doc(), and see which one passes all the unit tests. Of course, if both of

Re: Bug in BooleanScorer2

2006-06-21 Thread Chris Hostetter
: Not sure what the solution should be: I haven't looed at the code so i'm not sure ... but my first thought would be to try both this.doc() and BooleanScorer2.this.doc(), and see which one passes all the unit tests. Of course, if both of them pass the unit tests, then I'd be *really* worried.

Bug in BooleanScorer2

2006-06-21 Thread DM Smith
Hi, I have just tried to compile lucene with ecj, Eclipse's compiler, and it complains of errors with BooleanScorer2. The problematic construction is present 2x in the class: if (doc() > lastScoredDoc) { lastScoredDoc = doc(); coordinator.nrMatchers += super.nrMatchers;