Mailing-List: contact dev-help@apr.apache.org; run by ezmlm
Precedence: bulk
Date: Wed, 13 Aug 2003 02:08:57 -0400 (EDT)
From: Cliff Woolley <jwoolley@virginia.edu>
To: Greg Stein <gstein@lyra.org>
cc: dev@apr.apache.org
Subject: Re: [PATCH] bucket debugging.
In-Reply-To: <20030813053639.GA30695@lyra.org>
Message-ID: <Pine.GSO.4.53.0308130207420.26911@mamba.cs.Virginia.EDU>
References: <Pine.GSO.4.53.0308122315030.24781@mamba.cs.Virginia.EDU>
 <20030813053639.GA30695@lyra.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII

On Tue, 12 Aug 2003, Greg Stein wrote:

> > +    while (curr) {
> > +        if (node == curr) {
> > +            return 0;
> > +        }
> > +        curr = curr->next;
> > +    }
> > +    return 1;
>
> Forget the return code. Just stick an abort() in here.

return 0 to pop the assert, abort, same difference.  :)  But sure, I'll do
it that way, why not.