Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Sender: trawick@rdu26-58-158.nc.rr.com
To: Aaron Bannert <aaron@ebuilt.com>
Cc: dev@apr.apache.org
Subject: Re: cvs commit: apr/threadproc/unix thread.c
References: <20010801164548.45274.qmail@icarus.apache.org>
	<20010801141119.A5248@ebuilt.com>
From: Jeff Trawick <trawick@attglobal.net>
Date: 01 Aug 2001 17:25:55 -0400
In-Reply-To: <20010801141119.A5248@ebuilt.com>
Message-ID: <m38zh3tqu4.fsf@rdu26-58-158.nc.rr.com>
Lines: 23
User-Agent: Gnus/5.0808 (Gnus v5.8.8) Emacs/20.3
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

Aaron Bannert <aaron@ebuilt.com> writes:

> I posted a fix for this about a week and a half ago. The fix attempted
> to do the same thing on other platforms. Also, my fix made one extra
> change (that may or may not be agreeable to the group), and that was to
> change the prototype of apr_thread_exit() so it takes an apr_status_t
> instead of an (apr_status_t*). This was because the original method
> essentially required the return status (which is just an (int)) to be
> allocated from the heap merely so it could be passed back through
> thread_join() (or analog in all cases except os2). By making it a
> non-pointer the caller doesn't have to worry about where the memory for
> the (int) was allocated, because it's immediately copied.

That sounds better...  I remembered something being done with thread
status codes but assumed it had already been committed.

I'll try to find some time in the next 24hrs to look over the patch
and hopefully commit, assuming nobody voices any disagreements.

-- 
Jeff Trawick | trawick@attglobal.net | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...