Re: [PATCH] TPF responses

[Joe Moenich <mo...@us.ibm.com>]

Thanks to all for their comments.  We'll ensure everything is cleaned up before
committed.  Also, thanks for a great ApacheCon.

Joe Moenich
moenich@us.ibm.com
303 773-5483
tie-line 656-7487



new-httpd-owner@apache.org on 10/17/98 06:44:53 AM
Please respond to new-httpd@apache.org
To: new-httpd@apache.org
cc:
Subject: Re: [PATCH] TPF util_uri.c


Brian Behlendorf wrote:
>
> This is a comment that applies to this patch and other patches.  It's been
> generally agreed 'round these parts that type casts hide bugs, and thus
> should be avoided.  It's also been generally agreed that OS-specific
> sections should be kept out of the core code, to help keep it simpler.  So
> for example, your changes to GuessOS and ap_config.h are right; changes
> like the below, there should be a better way.
>
> >+ #ifdef TPF
> >+     struct hostent *hp = gethostbyname((char *)hostname);
> >+ #else
> >      struct hostent *hp = gethostbyname(hostname);
> >+ #endif /* TPF */

In some header:

#ifdef TPF
#define ap_os_gethostbyname(h) gethostbyname((char *)h)
#else
#define ap_os_gethostbyname(h) gethostbyname(h)
#endif

Cheers,

Ben.

--
Ben Laurie            |Phone: +44 (181) 735 0686| Apache Group member
Freelance Consultant  |Fax:   +44 (181) 735 0689|http://www.apache.org/
and Technical Director|Email: ben@algroup.co.uk |
A.L. Digital Ltd,     |Apache-SSL author     http://www.apache-ssl.org/
London, England.      |"Apache: TDG" http://www.ora.com/catalog/apache/

WE'RE RECRUITING! http://www.aldigital.co.uk/