Mailing-List: contact new-httpd-help@apache.org; run by ezmlm
Precedence: bulk
Reply-To: new-httpd@apache.org
Date: Wed, 17 May 2000 07:15:56 -0400 (EDT)
From: rbb@covalent.net
To: new-httpd@apache.org
Subject: Re: cvs commit: apache-2.0/src/lib/apr/network_io/unix sockets.c 
In-Reply-To: <200005162100.aa16190@gremlin-relay.ics.uci.edu>
Message-ID: <Pine.LNX.4.21.0005170714390.998-100000@koj.rkbloom.net>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII


> >   #ifdef BEOS
> >       return shutdown(thesocket->socketdes, how);
> >  -#endif
> >  +#else
> >       if (shutdown(thesocket->socketdes, how) == 0) {
> >           return APR_SUCCESS;
> >       }
> >       else {
> >           return errno;
> >       }
> >  +#endif
> >   }
> 
> I love this snippet -- it was part of the diff I was looking at while
> composing that long e-mail rant.
> 
> Does shutdown on BEOS return errno on error, or -1 (as is defined by BSD)?
> Anybody have a pointer to a web site with on-line BeOS man pages?
> It claims to be POSIX, so my guess is that this is just a bug.  The code
> should be simply
> 
>     return shutdown(thesocket->socketdes, how) ? errno : APR_SUCCESS;
> 
> I'll commit it.

I was trying to be VERY concervative while making these changes
yesterday.  I was assuming that most of the bugs in the BeOS code were
because code hadn't been brought up to date, not because the code was
wrong.

I would prefer to add a comment asking if this is right or not.  I think I
did that in one case.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------