From: Jonathan Tan <jonathantanmy@google.com>
Date: Fri, 26 May 2023 21:06:33 GMT
Subject: Re: [PATCH v2 1/7] strbuf.h: move declarations for strbuf.c functions from git-compat-util.h
Message-ID: <20230526210633.2290844-1-jonathantanmy@google.com>
In-Reply-To: <20230523192949.1271671-1-calvinwan@google.com>

Calvin Wan <calvinwan@google.com> writes:
> While functions like starts_with() probably should not belong in the
> boundaries of the strbuf library, this commit focuses on first splitting
> out headers from git-compat-util.h.
> 
> Signed-off-by: Calvin Wan <calvinwan@google.com>

I know we don't follow the 50-character rule for commit message headers
all the time, but I don't think it's necessary to break it here for
a relatively simple-to-explain change. Maybe "git-compat-util: move
strbuf.c funcs to its header". Same for the other patches.

The patch itself looks good.