Yes, it was wrong.Because we need to add the wdf for each document in the
RSet that contains the term. I will work on writing a regression test case
for this, sounds like a good idea.


On Mon, Aug 26, 2013 at 10:45 AM, Olly Betts <notifications@github.com>wrote:

> In xapian-core/expand/expandweight.h:
>
> >  	// get a non-zero weight.
> >  	if (wdf == 0) wdf = 1;
> > +	++rtermfreq;
> > +	rcollection_freq += wdf;
>
> This seems to suggest that the previously merged change for calculating
> rcollection_freq was wrong - that really deserves a regression testcase in
> the testsuite.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/xapian/xapian/pull/24/files#r5971333>
> .
>