Re: two agent nits

[Simon Josefsson <jas@extundo.com>]

Kevin Greiner <kgreiner@xpediantsolutions.com> writes:

> I finally realized what bothered me with this patch.  If you look
> closely, the resulting code has a cond statement with four cases.  In
> three cases, the result is simply nil.  In the fourth case, the
> function recurses.  Taken together that means that the function simply
> degenerates into nil.

Oops.

> So here's my version of the patch.  It is unfortunately a good deal
> more complex.  I'd appreciate some feedback before I check it in.

I did not read the patch, but I applied it and invoked `J s' and it
has been churning along for a few hours.  It made me realize 1 GB
isn't enough memory in a desktop machine these days, but other than
that it seems to work.

Once I have everything downloaded I'll start to use it for real.

Thanks.