apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: [PATCH] #39289 trylock test suite
Date Wed, 26 Apr 2006 12:35:33 GMT
On Thu, Apr 20, 2006 at 10:46:42AM -0400, Chris Darroch wrote:
>    I thought I'd ping the list to see if anyone cares to commit this
> patch, which just adds to the APR test suite some tests using
> apr_proc_mutex_trylock().  The only "real" patch is for testprocmutex.c.
> 
> http://issues.apache.org/bugzilla/show_bug.cgi?id=39289

Thanks Chris.  For stuff like this feel free to avoid the overhead of 
bugzilla; it's not a bug in the code which needs tracking as such, and 
it's easier to review patches in mail.

I committed the testprocmutex.c patch with a few changes:

- remove unnecessary whitespace changes
- use a function paramater rather than a global variable

haven't looked at the other patch.

Regards,

joe

Mime
View raw message