You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by Guenter Knauf <ef...@gmx.net> on 2004/03/15 18:29:22 UTC

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Hi Henri,
> hgomez@apache.org wrote:

>> hgomez      2004/03/01 05:47:23
>>
>>   Modified:    jk/native/common jk_ajp_common.c
>>   Log:
>>   More debug/trace infos on remote tomcats

> I added more infos on IP/PORT of remote tomcats in jk, since I've some
> reports that there is a problem for admins to determine which tomcat
> is down when they have many workers defined...

> I overcome the inet_ntoa by using a jk_dump_hinfo function which should
> works also in multi-threaded env.

> It should works on Unixes, but I'd like to have reports from
> Win32, Netware and others exotics OS users...
the change of 
--- jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 08:45:48 1.17
+++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01 13:37:38 1.18

breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
the problem is that in_addr_t and in_port_t are not defined.
Futhermore I consider in_addr_t as a dangerous var, google a bit and you will see that some OSes define it as unsigned long (which is what we need here) while others define it to a struct in_addr. NetWare also defines it to the in_addr struct so that its useless here - and even more ugly it would be if I had to include netinet/in.h for other reasons, then I would have to undef at least in_addr_t.
So we should avoid these typedefs at all, and simply use ulong and ushort.

--- jk_connect.c.orig	Mon Mar 15 16:04:08 2004
+++ jk_connect.c	      Mon Mar 15 18:16:28 2004
@@ -288,8 +288,8 @@
  */
 char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
 {
-	in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
-	in_port_t lport = htons(saddr->sin_port);
+	unsigned long laddr = htonl(saddr->sin_addr.s_addr);
+	unsigned short lport = htons(saddr->sin_port);
 
 	sprintf(buf, "%d.%d.%d.%d:%d", 
 	        (int)(laddr >> 24), (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr & 0xff), (int)lport);

I've not tested this yet, but at least I can now compile again. In addtion our older clib has a strange behavior with these functions, so there's certainly another additional patch needed for apache-1.3 and netscape; apache-2 should work with the above.

btw: we should really take care of tabs and avoid them - as outlined in the ASF developer docs. Currently there are a couple of files which contain tabs, they should be removed IMO.

Guenter.

 

---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by Henri Gomez <hg...@apache.org>.
jean-frederic clere wrote:
> Henri Gomez wrote:
> 
>> Guenter Knauf wrote:
>>
>>> Hi Henri,
>>>
>>>> hgomez@apache.org wrote:
>>>
>>>
>>>
>>>
>>>>> hgomez      2004/03/01 05:47:23
>>>>>
>>>>>  Modified:    jk/native/common jk_ajp_common.c
>>>>>  Log:
>>>>>  More debug/trace infos on remote tomcats
>>>
>>>
>>>
>>>
>>>> I added more infos on IP/PORT of remote tomcats in jk, since I've some
>>>> reports that there is a problem for admins to determine which tomcat
>>>> is down when they have many workers defined...
>>>
>>>
>>>
>>>
>>>> I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>>>> works also in multi-threaded env.
>>>
>>>
>>>
>>>
>>>> It should works on Unixes, but I'd like to have reports from
>>>> Win32, Netware and others exotics OS users...
>>>
>>>
>>>
>>> the change of --- 
>>> jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 
>>> 08:45:48 1.17
>>> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 
>>> 2004/03/01 13:37:38 1.18
>>>
>>> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
>>> the problem is that in_addr_t and in_port_t are not defined.
>>> Futhermore I consider in_addr_t as a dangerous var, google a bit and 
>>> you will see that some OSes define it as unsigned long (which is what 
>>> we need here) while others define it to a struct in_addr. NetWare 
>>> also defines it to the in_addr struct so that its useless here - and 
>>> even more ugly it would be if I had to include netinet/in.h for other 
>>> reasons, then I would have to undef at least in_addr_t.
>>> So we should avoid these typedefs at all, and simply use ulong and 
>>> ushort.
>>>
>>> --- jk_connect.c.orig    Mon Mar 15 16:04:08 2004
>>> +++ jk_connect.c          Mon Mar 15 18:16:28 2004
>>> @@ -288,8 +288,8 @@
>>>   */
>>>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>>>  {
>>> -    in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
>>> -    in_port_t lport = htons(saddr->sin_port);
>>> +    unsigned long laddr = htonl(saddr->sin_addr.s_addr);
>>> +    unsigned short lport = htons(saddr->sin_port);
>>>  
>>>      sprintf(buf, "%d.%d.%d.%d:%d",              (int)(laddr >> 24), 
>>> (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr 
>>> & 0xff), (int)lport);
>>>
>>> I've not tested this yet, but at least I can now compile again. In 
>>> addtion our older clib has a strange behavior with these functions, 
>>> so there's certainly another additional patch needed for apache-1.3 
>>> and netscape; apache-2 should work with the above.
>>>
>>> btw: we should really take care of tabs and avoid them - as outlined 
>>> in the ASF developer docs. Currently there are a couple of files 
>>> which contain tabs, they should be removed IMO.
>>>
>>> Guenter.
>>
>>
>>
>>  From my Linux manpage we shoulld use uint32_t and uint16_t
>> for htonl and htons.
> 
> 
> Use that should prevent compilers beeing unhappy:
> +++
>  unsigned long laddr = (unsigned long) htonl(saddr->sin_addr.s_addr);
>  unsigned short lport = (unsigned short) htons(saddr->sin_port);
> +++

Done

---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by jean-frederic clere <jf...@fujitsu-siemens.com>.
Henri Gomez wrote:
> Guenter Knauf wrote:
> 
>> Hi Henri,
>>
>>> hgomez@apache.org wrote:
>>
>>
>>
>>>> hgomez      2004/03/01 05:47:23
>>>>
>>>>  Modified:    jk/native/common jk_ajp_common.c
>>>>  Log:
>>>>  More debug/trace infos on remote tomcats
>>
>>
>>
>>> I added more infos on IP/PORT of remote tomcats in jk, since I've some
>>> reports that there is a problem for admins to determine which tomcat
>>> is down when they have many workers defined...
>>
>>
>>
>>> I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>>> works also in multi-threaded env.
>>
>>
>>
>>> It should works on Unixes, but I'd like to have reports from
>>> Win32, Netware and others exotics OS users...
>>
>>
>> the change of --- 
>> jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 
>> 08:45:48 1.17
>> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01 
>> 13:37:38 1.18
>>
>> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
>> the problem is that in_addr_t and in_port_t are not defined.
>> Futhermore I consider in_addr_t as a dangerous var, google a bit and 
>> you will see that some OSes define it as unsigned long (which is what 
>> we need here) while others define it to a struct in_addr. NetWare also 
>> defines it to the in_addr struct so that its useless here - and even 
>> more ugly it would be if I had to include netinet/in.h for other 
>> reasons, then I would have to undef at least in_addr_t.
>> So we should avoid these typedefs at all, and simply use ulong and 
>> ushort.
>>
>> --- jk_connect.c.orig    Mon Mar 15 16:04:08 2004
>> +++ jk_connect.c          Mon Mar 15 18:16:28 2004
>> @@ -288,8 +288,8 @@
>>   */
>>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>>  {
>> -    in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
>> -    in_port_t lport = htons(saddr->sin_port);
>> +    unsigned long laddr = htonl(saddr->sin_addr.s_addr);
>> +    unsigned short lport = htons(saddr->sin_port);
>>  
>>      sprintf(buf, "%d.%d.%d.%d:%d",              (int)(laddr >> 24), 
>> (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr & 
>> 0xff), (int)lport);
>>
>> I've not tested this yet, but at least I can now compile again. In 
>> addtion our older clib has a strange behavior with these functions, so 
>> there's certainly another additional patch needed for apache-1.3 and 
>> netscape; apache-2 should work with the above.
>>
>> btw: we should really take care of tabs and avoid them - as outlined 
>> in the ASF developer docs. Currently there are a couple of files which 
>> contain tabs, they should be removed IMO.
>>
>> Guenter.
> 
> 
>  From my Linux manpage we shoulld use uint32_t and uint16_t
> for htonl and htons.

Use that should prevent compilers beeing unhappy:
+++
  unsigned long laddr = (unsigned long) htonl(saddr->sin_addr.s_addr);
  unsigned short lport = (unsigned short) htons(saddr->sin_port);
+++

> 
> What about Netware ?
> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org
> 
> 


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by Henri Gomez <hg...@apache.org>.
Henri Gomez wrote:

> Guenter Knauf wrote:
> 
>> Hi Henri,
>>
>>> hgomez@apache.org wrote:
>>
>>
>>
>>>> hgomez      2004/03/01 05:47:23
>>>>
>>>>  Modified:    jk/native/common jk_ajp_common.c
>>>>  Log:
>>>>  More debug/trace infos on remote tomcats
>>
>>
>>
>>> I added more infos on IP/PORT of remote tomcats in jk, since I've some
>>> reports that there is a problem for admins to determine which tomcat
>>> is down when they have many workers defined...
>>
>>
>>
>>> I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>>> works also in multi-threaded env.
>>
>>
>>
>>> It should works on Unixes, but I'd like to have reports from
>>> Win32, Netware and others exotics OS users...
>>
>>
>> the change of --- 
>> jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 
>> 08:45:48 1.17
>> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01 
>> 13:37:38 1.18
>>
>> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
>> the problem is that in_addr_t and in_port_t are not defined.
>> Futhermore I consider in_addr_t as a dangerous var, google a bit and 
>> you will see that some OSes define it as unsigned long (which is what 
>> we need here) while others define it to a struct in_addr. NetWare also 
>> defines it to the in_addr struct so that its useless here - and even 
>> more ugly it would be if I had to include netinet/in.h for other 
>> reasons, then I would have to undef at least in_addr_t.
>> So we should avoid these typedefs at all, and simply use ulong and 
>> ushort.
>>
>> --- jk_connect.c.orig    Mon Mar 15 16:04:08 2004
>> +++ jk_connect.c          Mon Mar 15 18:16:28 2004
>> @@ -288,8 +288,8 @@
>>   */
>>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>>  {
>> -    in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
>> -    in_port_t lport = htons(saddr->sin_port);
>> +    unsigned long laddr = htonl(saddr->sin_addr.s_addr);
>> +    unsigned short lport = htons(saddr->sin_port);
>>  
>>      sprintf(buf, "%d.%d.%d.%d:%d",              (int)(laddr >> 24), 
>> (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr & 
>> 0xff), (int)lport);
>>
>> I've not tested this yet, but at least I can now compile again. In 
>> addtion our older clib has a strange behavior with these functions, so 
>> there's certainly another additional patch needed for apache-1.3 and 
>> netscape; apache-2 should work with the above.
>>
>> btw: we should really take care of tabs and avoid them - as outlined 
>> in the ASF developer docs. Currently there are a couple of files which 
>> contain tabs, they should be removed IMO.
>>
>> Guenter.
> 
> 
>  From my Linux manpage we shoulld use uint32_t and uint16_t
> for htonl and htons.
> 
> What about Netware ?

For now I'm usin unsigned long and unsigned short

---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by Henri Gomez <hg...@apache.org>.
Guenter Knauf wrote:

> Hi Henri,
> 
>>hgomez@apache.org wrote:
> 
> 
>>>hgomez      2004/03/01 05:47:23
>>>
>>>  Modified:    jk/native/common jk_ajp_common.c
>>>  Log:
>>>  More debug/trace infos on remote tomcats
> 
> 
>>I added more infos on IP/PORT of remote tomcats in jk, since I've some
>>reports that there is a problem for admins to determine which tomcat
>>is down when they have many workers defined...
> 
> 
>>I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>>works also in multi-threaded env.
> 
> 
>>It should works on Unixes, but I'd like to have reports from
>>Win32, Netware and others exotics OS users...
> 
> the change of 
> --- jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 08:45:48 1.17
> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01 13:37:38 1.18
> 
> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
> the problem is that in_addr_t and in_port_t are not defined.
> Futhermore I consider in_addr_t as a dangerous var, google a bit and you will see that some OSes define it as unsigned long (which is what we need here) while others define it to a struct in_addr. NetWare also defines it to the in_addr struct so that its useless here - and even more ugly it would be if I had to include netinet/in.h for other reasons, then I would have to undef at least in_addr_t.
> So we should avoid these typedefs at all, and simply use ulong and ushort.
> 
> --- jk_connect.c.orig	Mon Mar 15 16:04:08 2004
> +++ jk_connect.c	      Mon Mar 15 18:16:28 2004
> @@ -288,8 +288,8 @@
>   */
>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>  {
> -	in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
> -	in_port_t lport = htons(saddr->sin_port);
> +	unsigned long laddr = htonl(saddr->sin_addr.s_addr);
> +	unsigned short lport = htons(saddr->sin_port);
>  
>  	sprintf(buf, "%d.%d.%d.%d:%d", 
>  	        (int)(laddr >> 24), (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr & 0xff), (int)lport);
> 
> I've not tested this yet, but at least I can now compile again. In addtion our older clib has a strange behavior with these functions, so there's certainly another additional patch needed for apache-1.3 and netscape; apache-2 should work with the above.
> 
> btw: we should really take care of tabs and avoid them - as outlined in the ASF developer docs. Currently there are a couple of files which contain tabs, they should be removed IMO.
> 
> Guenter.

 From my Linux manpage we shoulld use uint32_t and uint16_t
for htonl and htons.

What about Netware ?




---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: SubTopic source code Tabs - part2

Posted by NormW <no...@bocnet.com.au>.
Greetings All.
Find at the links below the last Tab's I've found. Please note the mod_jk2.c
referred to is the \server\Apache13 version.

\native2\common
http://www.gknw.com/norm/jk_channel_un.c.diff
\native2\server\apache13
http://www.gknw.com/norm/mod_jk2.c.diff

Norm


---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: SubTopic - Removal of Tabs

Posted by NormW <no...@bocnet.com.au>.
Good morning All.
The following links to diff's for current Mod_Jk2 source files should remove
99% of Tabs. Will do the remainder 'soon'.
\native2\common
http://www.gknw.com/norm/jk_channel.c.diff
http://www.gknw.com/norm/jk_channel_jni.c.diff
http://www.gknw.com/norm/jk_config.c.diff
http://www.gknw.com/norm/jk_handler_logon.c.diff
http://www.gknw.com/norm/jk_handler_response.c.diff
http://www.gknw.com/norm/jk_msg_ajp.c.diff
http://www.gknw.com/norm/jk_nwmain.c.diff
http://www.gknw.com/norm/jk_registry.h.diff
http://www.gknw.com/norm/jk_requtil.c.diff
http://www.gknw.com/norm/jk_user.c.diff
http://www.gknw.com/norm/jk_workerEnv.c.diff
\native2\server\aolserver
http://www.gknw.com/norm/jk_logger_ns.c.diff
http://www.gknw.com/norm/jk_service_ns.c.diff
http://www.gknw.com/norm/nsjk2.c.diff
\native2\server\dsapi
http://www.gknw.com/norm/config.h.diff
http://www.gknw.com/norm/dsapifilter.h.diff
http://www.gknw.com/norm/jk_dsapi_plugin.c.diff

Norm


---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by jean-frederic clere <jf...@fujitsu-siemens.com>.
Guenter Knauf wrote:
> Hi Henri,
> 
>>hgomez@apache.org wrote:
> 
> 
>>>hgomez      2004/03/01 05:47:23
>>>
>>>  Modified:    jk/native/common jk_ajp_common.c
>>>  Log:
>>>  More debug/trace infos on remote tomcats
> 
> 
>>I added more infos on IP/PORT of remote tomcats in jk, since I've some
>>reports that there is a problem for admins to determine which tomcat
>>is down when they have many workers defined...
> 
> 
>>I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>>works also in multi-threaded env.
> 
> 
>>It should works on Unixes, but I'd like to have reports from
>>Win32, Netware and others exotics OS users...
> 
> the change of 
> --- jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24 08:45:48 1.17
> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01 13:37:38 1.18
> 
> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
> the problem is that in_addr_t and in_port_t are not defined.
> Futhermore I consider in_addr_t as a dangerous var, google a bit and you will see that some OSes define it as unsigned long (which is what we need here) while others define it to a struct in_addr. NetWare also defines it to the in_addr struct so that its useless here - and even more ugly it would be if I had to include netinet/in.h for other reasons, then I would have to undef at least in_addr_t.
> So we should avoid these typedefs at all, and simply use ulong and ushort.
> 
> --- jk_connect.c.orig	Mon Mar 15 16:04:08 2004
> +++ jk_connect.c	      Mon Mar 15 18:16:28 2004
> @@ -288,8 +288,8 @@
>   */
>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>  {
> -	in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
> -	in_port_t lport = htons(saddr->sin_port);
> +	unsigned long laddr = htonl(saddr->sin_addr.s_addr);
> +	unsigned short lport = htons(saddr->sin_port);
>  
>  	sprintf(buf, "%d.%d.%d.%d:%d", 
>  	        (int)(laddr >> 24), (int)((laddr >> 16) & 0xff), (int)((laddr >> 8) & 0xff), (int)(laddr & 0xff), (int)lport);
> 
> I've not tested this yet, but at least I can now compile again. In addtion our older clib has a strange behavior with these functions, so there's certainly another additional patch needed for apache-1.3 and netscape; apache-2 should work with the above.
> 
> btw: we should really take care of tabs and avoid them - as outlined in the ASF developer docs. Currently there are a couple of files which contain tabs, they should be removed IMO.
>

Yep.

> Guenter.
> 
>  
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org
> 
> 
> 


Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_connect.c

Posted by Guenter Knauf <ef...@gmx.net>.
Hi Henri,
just tested that it breaks compilation on Win32 platform to -- my patch solved it.

Guenter.

>> hgomez@apache.org wrote:

>>> hgomez      2004/03/01 05:47:23
>>>
>>>   Modified:    jk/native/common jk_ajp_common.c
>>>   Log:
>>>   More debug/trace infos on remote tomcats

>> I added more infos on IP/PORT of remote tomcats in jk, since I've some
>> reports that there is a problem for admins to determine which tomcat
>> is down when they have many workers defined...

>> I overcome the inet_ntoa by using a jk_dump_hinfo function which should
>> works also in multi-threaded env.

>> It should works on Unixes, but I'd like to have reports from
>> Win32, Netware and others exotics OS users...
> the change of
> --- jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/02/24
> 08:45:48 1.17
> +++ jakarta-tomcat-connectors/jk/native/common/jk_connect.c 2004/03/01
> 13:37:38 1.18

> breaks all targets on NetWare: netscape, apache-1.3 and apache-2;
> the problem is that in_addr_t and in_port_t are not defined.
> Futhermore I consider in_addr_t as a dangerous var, google a bit and you
> will see that some OSes define it as unsigned long (which is what we need
> here) while others define it to a struct in_addr. NetWare also defines it
> to the in_addr struct so that its useless here - and even more ugly it
> would be if I had to include netinet/in.h for other reasons, then I would
> have to undef at least in_addr_t.
> So we should avoid these typedefs at all, and simply use ulong and ushort.

> --- jk_connect.c.origMon Mar 15 16:04:08 2004
> +++ jk_connect.c      Mon Mar 15 18:16:28 2004
> @@ -288,8 +288,8 @@
>   */
>  char * jk_dump_hinfo(struct sockaddr_in *saddr, char * buf)
>  {
> -in_addr_t laddr = htonl(saddr->sin_addr.s_addr);
> -in_port_t lport = htons(saddr->sin_port);
> +unsigned long laddr = htonl(saddr->sin_addr.s_addr);
> +unsigned short lport = htons(saddr->sin_port);

>  sprintf(buf, "%d.%d.%d.%d:%d",
>          (int)(laddr >> 24), (int)((laddr >> 16) & 0xff), (int)((laddr >>
>          8) & 0xff), (int)(laddr & 0xff), (int)lport);

> I've not tested this yet, but at least I can now compile again. In addtion
> our older clib has a strange behavior with these functions, so there's
> certainly another additional patch needed for apache-1.3 and netscape;
> apache-2 should work with the above.

> btw: we should really take care of tabs and avoid them - as outlined in
> the ASF developer docs. Currently there are a couple of files which
> contain tabs, they should be removed IMO.

> Guenter.



> ---------------------------------------------------------------------
> To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org