On Thursday 15 May 2014 10:03:25, Ruediger Pluem wrote:
> > +    if (old == 2 && pool->in_use_by == apr_os_thread_current()) {
> 
> How can old ever be 2?

That was a leftover from a version with a separate state for the pool 
being cleared.

> 
> > +        /* cleanups may use the pool */
> > +        return;
> > +    }
> > +
> > +    if (old != 0)
> > +        pool_concurrency_abort(pool, __func__, old);
> 
> Isn't __func__ C99?


Thanks for the review. Both issues fixed in r1595549.

Stefan

Reply via email to