From: Jeff King Date: Mon, 14 Jan 2019 15:40:19 GMT Subject: Re: [PATCH 3/5] match-trees: use hashcpy to splice trees Message-ID: <20190114154019.GA4202@sigill.intra.peff.net> In-Reply-To: <20190114013017.GP423984@genre.crustytoothpaste.net> On Mon, Jan 14, 2019 at 01:30:17AM +0000, brian m. carlson wrote: > > > So the "most correct" thing is probably something like this: > > > > Of course, it would be nice if what I sent compiled. ;) > > > > rewrite_here does double duty: it's the pointer in the tree that we need > > to rewrite, and it's also the oid we pass down recursively. So we have > > to handle both cases, like so: > > Since I took most of the patch you wrote, may I apply your sign-off to > the resulting patch I send out? Yes, definitely. -Peff