Re: SQL indices a WC format bump and 1.7

On Fri, Sep 2, 2011 at 18:32, Peter Samuelson <peter_at_p12n.org> wrote:
>
> [Greg Stein]
>> Since we are selecting only "current nodes" for each local_relpath,
>> then I presume we don't need the apr_hash_get() in there, correct?
>
> Yeah, I confirmed with him on IRC that the wc_db.c change (chiefly
> deleting the extraneous apr_hash_get() call) is a cleanup that could be
> applied separately.  I could go either way.  It is a micro-optimization
> on a set of changes whose original purpose was optimization.

Oh! Whoops. My diff was backwards. I thought the apr_hash_get() was
being *added* ... but that's not the case.

Sure, this little optimization is just fine.

Cheers,
-g
Received on 2011-09-03 01:05:43 CEST