[Xapian-devel] Merging of the TfIdf patch

[aarsh shah]

No Olly,there is no specific reason for that .I just decided to go with
uppercase strings as that makes them  easily visible and conspicuous when
reading the documentation and also while reading the code.However, the case
wont make any difference to the code.

-Regards
-Aarsh


On Tue, Apr 2, 2013 at 8:39 AM, Olly Betts <olly at survex.com> wrote:

> On Tue, Mar 26, 2013 at 09:29:53PM +0530, aarsh shah wrote:
> > Hello Guys. I have updated the code,tests,documentation,makefile entries
> > and the registry entry of the* *TfIdf patch as per the feedback.Please do
> > let me know if any additional changes are required before the  patch can
> be
> > merged,
>
> One question - the 3 character codes that TfIdfWeight takes are (I
> assume) modelled on those used by SMART:
>
> http://people.csail.mit.edu/jrennie/ecoc-svm/smart.html
>
> But TfIdfWeight requires them to be in upper-case, while SMART documents
> them as being lowercase.  Is there a reason for that?
>
> Cheers,
>     Olly
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.xapian.org/pipermail/xapian-devel/attachments/20130403/cbb26b9b/attachment.htm>