Re: [PATCH v9 1/1] http: Add Accept-Language header if possible

From: Jeff King <peff@peff.net>
Date: Feb 26, 2015, 22:45 UTC
Message-ID: <20150226224558.GA10311@peff.net>
In-Reply-To: <20150226223603.GA27946@peff.net>

On Thu, Feb 26, 2015 at 05:36:03PM -0500, Jeff King wrote:

Show 10 quoted lines

> > [-Wstrict-prototypes]
> >  const char *get_preferred_languages();
> >  ^
> 
> Hmph. The compiler is right that it should be:
> 
>  const char *get_preferred_languages(void);
> 
> but my gcc (4.9.2, with -Wstrict_prototypes) does not seem to notice it!
> Weird.

Ugh. I have a snippet in my config.mak that relaxes the warnings on older versions of git, and it was accidentally triggering due to a typo. :(

So that explains that. Junio, do you mind squashing in:

diff --git a/gettext.h b/gettext.h
index 5d8d2df..33696a4 100644
--- a/gettext.h
+++ b/gettext.h
@@ -89,6 +89,6 @@ const char *Q_(const char *msgid, const char *plu, unsigned long n)
 #define N_(msgid) (msgid)
 #endif
 
-const char *get_preferred_languages();
+const char *get_preferred_languages(void);
 
 #endif