"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

From:
Stefan Sperling <stsp@stsp.name>
Subject:
Re: got: add diffstat to got diffg
To:
Mark Jamsek <mark@jamsek.com>
Cc:
Omar Polo <op@omarpolo.com>, Game of Trees <gameoftrees@openbsd.org>
Date:
Mon, 9 Jan 2023 15:58:44 +0100

Download raw body.

Thread
On Tue, Jan 10, 2023 at 12:11:12AM +1100, Mark Jamsek wrote:
> I've made all suggested changes except for moving 'path' into function
> scope because (1) with your other brilliant suggestion to allocate
> 'change' in get_diffstat() (I can't believe I didn't think of that!), we
> now only have one free(path) on failure; and (2) I actually have a diff
> in my tree making some minor changes to diff_blobs() doing much the same
> as you mentioned regarding failure leaks. I didn't want to mix that in
> with this diff though. So I might hoist path up to function scope in
> that along with the other cleanup.
> 
> Thanks, op!

Looks fine to me, ok.