From: Christian Couder Date: Tue, 22 May 2012 20:30:45 GMT Subject: Re: [PATCH 2/7] sequencer: release a strbuf used in save_head() Message-ID: <20120522.223045.876953043846314069.chriscool@tuxfamily.org> In-Reply-To: <20120522141735.GA6660@burratino> From: Jonathan Nieder > >> + strbuf_release(&buf); > > Makes good sense. Actually, I'm not sure why this allocation is > needed in the first place. Would something like the following work? Ok, I will use your patch in the next iteration. Thanks, Christian.