From: Junio C Hamano Date: Fri, 10 May 2024 21:19:25 GMT Subject: Re: [PATCH v2 03/11] reftable/writer: drop static variable used to initialize strbuf Message-ID: In-Reply-To: Patrick Steinhardt writes: > We have a static variable in the reftable writer code that is merely > used to initialize the `last_key` of the writer. Convert the code to > instead use `strbuf_init()` and drop the variable. Nice. There is no guarantee that a structure assignment of a just-initialized empty strbuf to another will stay to be a trouble free operation, and using an explicit _init() call is the right thing to do.