On 10/1/05, Garrett Rooney <[EMAIL PROTECTED]> wrote:
> On 10/1/05, Jeff Trawick <[EMAIL PROTECTED]> wrote:
> > Purify on Solaris pointed out that we were referencing uninitialized
> > storage when checking revents.  It was uninitialized (by kernel/libc)
> > because no conditions were met.
> >
> > Concerns?
>
> The comment should be "caller should look at revents", not
> "rtnevents".

Actually I meant the caller of apr_poll().  But that sort of guidance
is better placed in the documentation (which I'll do).

>                               Also, it seems like you could
> compare against i, not *nsds, and avoid having to dereference a
> pointer

will-do

Thanks for the comments!

Reply via email to