On Fri, 25 Aug 2000, Quim Sanmarti wrote:
> OK -- there's also a newer version, 1.22.2.6, I guess I should use that.
The differences between 1.22.2.5 and 1.22.2.6 are fairly minimal. The
patch program is usually pretty good about ignoring small differences and
I usually take care of the other problems. The catch here was that it
wasn't clear the best way to proceed, esp. since I didn't write either bit
of code. :-)
> Concerning Arthur's patch... it will not work if the first word in the
> phrase has isIgnore set. I'd rather keep my version, that includes also the
> optimization that stops lookups if some two words in a phrase are nowhere
> found to be contiguous.
Sure.
> On the other hand, I must withdraw the other modification in perform_phrase
> that concerned ignored words in a phrase. It's plain wrong, since words
> marked to be ignored are not indexed, so the location counter is not
> incremented by them (*). Sorry, my tests were bad, I shouldn't have posted
> that modification.
I wondered about that and was going to ignore that section when merging in
the patch. But I can't say I haven't made mistakes in parser.cc either.
:-)
As far as a "near" operator, I gave up all hope of adding it to parser.cc
and would much rather replace the whole mess so we can add "near" in a
sensible way. (Which may or may not be ParseTree, though you no longer
voice objections to the scheme.)
-Geoff
------------------------------------
To unsubscribe from the htdig3-dev mailing list, send a message to
[EMAIL PROTECTED]
You will receive a message to confirm this.
