You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by David Robinson <dr...@ast.cam.ac.uk> on 1996/02/07 17:55:00 UTC

82.setsockopt_next.patch

>From: Martijn Koster <ma...@beach.webcrawler.com>
>Subject: setsockopt doesn't take a const declared argument for NeXT
>Affects: http_main.c
>Changelog: setsockopt no longer takes a const declared argument when
>        compiling under NeXT.
>
>*** http_main.c.dist    Sun Jan 21 13:24:54 1996
>--- http_main.c Sun Jan 21 13:32:04 1996
>***************
>*** 904,918 ****
...
>--- 904,928 ----
...
>+ #ifdef NEXT  
>+     if((setsockopt(sd,SOL_SOCKET,SO_KEEPALIVE,(void*)&keepalive_value,
>+         sizeof(keepalive_value))) == -1) {
>+ #else  
>      if((setsockopt(sd,SOL_SOCKET,SO_KEEPALIVE,(const void *)&keepalive_value,
>          sizeof(keepalive_value))) == -1) {
>+ #endif
>          fprintf(stderr,"httpd: could not set socket option SO_KEEPALIVE\n"); 
>          perror("setsockopt"); 
>          exit(1); 

Isn't the change correct for all architectures, not just the NeXT?
If so, then the #ifdefs should be removed.

 David.