Re: [PATCH] builtin/: add UNLEAKs
- Date: Mon, 2 Oct 2017 12:20:56 +0200
- From: Martin Ågren <martin.agren@xxxxxxxxx>
- Subject: Re: [PATCH] builtin/: add UNLEAKs
On 2 October 2017 at 08:25, Jeff King <peff@xxxxxxxx> wrote:
> On Sun, Oct 01, 2017 at 07:42:08PM +0200, Martin Ågren wrote:
>> Add some UNLEAKs where we are about to return from `cmd_*`. UNLEAK the
>> variables in the same order as we've declared them. While addressing
>> `msg` in builtin/tag.c, convert the existing `strbuf_release()` calls as
> It might have raised Junio's eyebrows less to say something like:
> ...convert the existing strbuf_release() calls as well (they're not
> wrong, but they also accomplish nothing and create an inconsistency
> with the UNLEAKed variables).
Most likely yes. Sorry about that. I have yet to be critiqued for
writing *too much* in a commit message. That should tell me something.
> Seeing hunks like this makes me happy with the UNLEAK() solution. It
> would have been a real pain to do this via actual freeing.
Yes, I was very happy to have it handy. :-)