apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Yossi Neiman <freeswi...@cartissolutions.com>
Subject Re: apr_queue_t enhancement - please review for inclusion
Date Wed, 03 Jan 2007 18:07:24 GMT
See below.

Ryan Phillips wrote:
> Yossi Neiman <freeswitch@cartissolutions.com> said:
<snip snip>
>> http://issues.apache.org/bugzilla/show_bug.cgi?id=41254
>>
> 
> I would find this patch useful.  One problem I see with the patch is
> that 'rv' is used uninitialized a few times:
> 
> {
>     apr_status_t rv;
> 
>     if (!queue->terminated) {
>         ...
>     }
> 
>     return rv;
> }
> 
> -ryan

How right you are.  I have fixed this error and have posted updated 
patches to the bugtracker (under the same issue of course).  I'm glad 
that this is of interest to you.

-Yossi

Mime
View raw message