Re: [PATCH v2 2/4] worktree: link worktrees with relative paths

From

Caleb White <cdwhite3@pm.me>

Date

Oct 7, 2024, 16:59 UTC

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!