[htdig-dev] Re: Changes to htsearch.cc version 1.60, 2002/12/30 12:42:59

[Lachlan A. <lh...@us...>]

Greetings Neal,

These conditions were subsumed into the earlier tests:
   if (boolean && mystrcasecmp(word.get(), "and") =3D=3D 0)
became
    if (boolean && (mystrcasecmp(word.get(), "+") =3D=3D 0
      || mystrcasecmp(word.get(), boolean_keywords[AND]) =3D=3D 0))
and similarly for "-".

To my mind this is clearer, and it also should produce (marginally)=20
smaller code.

I hope the porting is going well.  Is it going to change the main Unix=20
source, be a patch relative to the main source, or be a stand-alone=20
port?

Cheers,
Lachlan

On Thu, 5 Jun 2003 07:54, Neal Richter wrote:

> On 2002/12/30 12:42:59
> These lines were removed new line 550:
>
> else if (boolean && mystrcasecmp(word.get(), "+") =3D=3D 0)
> tempWords.Add(new WeightWord("&", -1.0));
> else if (boolean && mystrcasecmp(word.get(), "-") =3D=3D 0)
> tempWords.Add(new WeightWord("!", -1.0));
>
> Any explaination on this particular change?  This looks to
> implement + (required word) and - (exclude word).