From: Duy Nguyen <pclouds@gmail.com>
Date: Tue, 14 Feb 2017 09:43:49 GMT
Subject: Re: [PATCH 04/11] files-backend: replace *git_path*() with files_path()
Message-ID: <CACsJy8CeEZ9XESGaCLZs8uznw_XazjTmbumzo6foiGbUp6Qa5Q@mail.gmail.com>
In-Reply-To: <8f6e09aa-f578-6faf-6acc-e35be71ee990@ramsayjones.plus.com>

On Tue, Feb 14, 2017 at 3:58 AM, Ramsay Jones
<ramsay@ramsayjones.plus.com> wrote:
>> -void files_path(struct files_ref_store *refs, struct strbuf *sb,
>> -             const char *fmt, ...) __attribute__((format (printf, 3, 4)));
>
> You only added this in the last commit, so maybe mark it static in
> the previous patch! Also, just in case you were wondering, the 'Why?'
> of the previous email was, "Why do you need this forward declaration?"
> (hint: you don't ;-)

Yeah, my compiler considers unused static functions an error. But I
guess I should adjust those million compiler flags instead of doing it
like this. I'll need to check if it's an error with DEVELOPER_CFLAGS
because people may bisect with that.
-- 
Duy