On Fri, Sep 2, 2011 at 18:32, Peter Samuelson <pe...@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
