Elia Pinto <[email protected]> writes:
> Add the FORMATPRINTF macro for declaring the gcc function attribute 'format
> printf'
> for code style consistency with similar macro that git already use for other
> gcc
> attributes. And use it where necessary.
>
> Elia Pinto (22):
> git-compat-util.h: add the FORMATPRINTF macro
Hmm. Given that we already have
#ifndef __GNUC__
#ifndef __attribute__
#define __attribute__(x)
#endif
#endif
in git-compat-util.h, it is really between:
__attribute__((format (printf, 1, 2)))
void advise(const char *advice, ...);
__attribute__((format (printf,2,3)))
extern void strbuf_addf(struct strbuf *sb, const char *fmt, ...);
and
FORMATPRINTF(1,2)
void advise(const char *advice, ...);
FORMATPRINTF(2,3)
extern void strbuf_addf(struct strbuf *sb, const char *fmt, ...);
Perhaps I am biased for staring at our source code for too long, but
somehow the latter looks unnecessarily loud, spelled in all caps.
I dunno. What does this really buy us?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
