Martin Ågren <martin.agren@xxxxxxxxx> writes: > ... While addressing `msg` in builtin/tag.c, convert the existing > `strbuf_release()` calls as well. That part of this patch made me raise eyebrows a bit but only slightly. We are about to leave the function to exit anyway, and all the other that got new UNLEAK(var) at the end are doing the same thing. So overall I think this patch makes sense. Thanks.
