From: Caleb White <cdwhite3@pm.me>
Date: Mon, 07 Oct 2024 16:59:21 GMT
Subject: Re: [PATCH v2 2/4] worktree: link worktrees with relative paths
Message-ID: <XhMvjO_NVmnkqWATQ8dMn0D7igePOHd7t0zfhKOr8ZTzd90-zslpej19uaEhtjpZi0nT1JalpyViubAEBBc5VRq6YI-AsrTftI180UJWc7c=@pm.me>
In-Reply-To: <ZwNZRX1LHlxawJJc@ArchLinux>

On Sunday, October 6th, 2024 at 22:45, shejialuo <shejialuo@gmail.com> wrote:
> You don't need to create a new "strbuf" for each of the paths. You could
> just use "strbuf_reset" for only one "struct strbuf".
> 

> struct strbuf file = STRBUF_INIT;
> 

> if (...) {
> strbuf_addf(...);
> strbuf_reset(&file);
> strbuf_addf(...);
> 

> }
> 

> strbuf_release(&file);

Ah, this is a better design---I've updated this to use a single strbuf, thanks!