Gilles Detillieux (email@example.com)
Fri, 12 Mar 1999 10:13:15 -0600 (CST)
According to Antti Rauramo:
> Geoff Hutchison wrote:
> > OK, here's a completely untested patch. As it turns out, someone's in the
> > process of upgrading my development box, so I can't compile. :-(
> > The first also takes care of a smallish memory leak in the display code.
> > The second should take care of the problems with empty descriptions. (I
> > noticed it checks for null strings, but not strings of nulls :-).
> > So once again, not only are these untested, but I didn't get a chance to
> > compile them, so I might have made some silly mistake somewhere. If so,
> > please let me know...
> Ookay, thank you very much once again, here's the little silly mistake
> came up in compiling:
> > *** htdig-3.1.1/htsearch/Display.cc Wed Feb 17 00:03:56 1999
> > --- htdig3/htsearch/Display.cc Wed Mar 10 13:57:06 1999
> > *************** Display::displayMatch(ResultMatch *match
> > --- 342,350 ----
> > ! *description = ((String*) (*list));
> I think this should be "description = ((String*) (*list));", without
> "*", that is.
> Or maybe I'm wrong, 'cause the patch seems to make another leak, or
> See http://www.edita.fi/cgi-bin/htdig/htsearch-descpatch?config=nm2, try
> searching for "iskusana".
Hmm, no, you definitely don't want to assign to description, because just
above that, description is set to point to a new String object. You must
assign to *description, so that the new String object receives the data
from the list entry. How about this?
*description = ((String*) (*list))->get();
either that, or just get rid of the temporary variable "description", and
do something like this:
vars.Add("DESCRIPTION", new String(((String*) (*list))->get()));
-- Gilles R. Detillieux E-mail: <firstname.lastname@example.org> Spinal Cord Research Centre WWW: http://www.scrc.umanitoba.ca/~grdetil Dept. Physiology, U. of Manitoba Phone: (204)789-3766 Winnipeg, MB R3E 3J7 (Canada) Fax: (204)789-3930 ------------------------------------ To unsubscribe from the htdig mailing list, send a message to email@example.com containing the single word "unsubscribe" in the SUBJECT of the message.
This archive was generated by hypermail 2.0b3 on Mon Mar 15 1999 - 08:57:47 PST