Re: [htdig3-dev] Re: iTools version of ht://Dig on MacOS X


Subject: Re: [htdig3-dev] Re: iTools version of ht://Dig on MacOS X
From: Gilles Detillieux (grdetil@scrc.umanitoba.ca)
Date: Wed Feb 16 2000 - 12:26:09 PST


According to Geoff Hutchison:
> On Wed, 16 Feb 2000, Gilles Detillieux wrote:
> > be to rename the method names that conflict with the socket library
> > functions, so that you can do away with the "::" overrides, and add a
>
> Yes, looking at their changes, I think this may be a good thing,
> especially for portability. Should we just use the Ht prefix or try to go
> with synonyms?

I was wondering if it wouldn't be sufficient to change the first letter to
a capital, e.g. Connection::accept() => Connection::Accept(), so that the
call to accept() in Connection.cc would unambiguously refer to the C library
accept() function, and likewise for the other conflicts.

> > test in the autoconf stuff for sys/malloc.h as well as malloc.h. Come
> > to think of it, why use malloc.h at at all? DB2_db::db_init() uses
> > calloc() to allocate and clear a DB_ENV object, but it should probably
> > use "new" and memset() instead, then you can do away with malloc.h.
>
> I'm pretty sure the GNU regex.c uses malloc. However, it seems to be
> pretty sophisticated about working out which it can use.

GNU regex.c uses malloc() et al., but doesn't include malloc.h. The iTools
changes didn't touch regex.c, so it seems it was OK. DB2_db.cc is C++
code, so it probably should use "new" rather than malloc() functions.

-- 
Gilles R. Detillieux              E-mail: <grdetil@scrc.umanitoba.ca>
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 htdig3-dev mailing list, send a message to htdig3-dev-unsubscribe@htdig.org You will receive a message to confirm this.



This archive was generated by hypermail 2b28 : Wed Feb 16 2000 - 12:29:15 PST