aboutsummaryrefslogtreecommitdiff
path: root/cache.h
diff options
context:
space:
mode:
authorMax Horn <max@quendi.de>2013-01-16 10:09:29 -0800
committerJunio C Hamano <gitster@pobox.com>2013-01-16 12:47:46 -0800
commit5ded807f7c0be10ecbb31555e5d10dee553752d6 (patch)
tree4a102e5f0d148fe587cedef9fe9eaa2b4120001e /cache.h
parenta469a1019352b8efc4bd7003b0bd59eb60fc428c (diff)
downloadgit-5ded807f7c0be10ecbb31555e5d10dee553752d6.tar.gz
git-5ded807f7c0be10ecbb31555e5d10dee553752d6.tar.xz
fix clang -Wunused-value warnings for error functions
Commit a469a10 wraps some error calls in macros to give the compiler a chance to do more static analysis on their constant -1 return value. We limit the use of these macros to __GNUC__, since gcc is the primary beneficiary of the new information, and because we use GNU features for handling variadic macros. However, clang also defines __GNUC__, but generates warnings with -Wunused-value when these macros are used in a void context, because the constant "-1" ends up being useless. Gcc does not complain about this case (though it is unclear if it is because it is smart enough to see what we are doing, or too dumb to realize that the -1 is unused). We can squelch the warning by just disabling these macros when clang is in use. Signed-off-by: Max Horn <max@quendi.de> Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'cache.h')
-rw-r--r--cache.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/cache.h b/cache.h
index 0e8e5d800..98b62bcb5 100644
--- a/cache.h
+++ b/cache.h
@@ -1136,7 +1136,7 @@ extern int check_repository_format_version(const char *var, const char *value, v
extern int git_env_bool(const char *, int);
extern int git_config_system(void);
extern int config_error_nonbool(const char *);
-#ifdef __GNUC__
+#if defined(__GNUC__) && ! defined(__clang__)
#define config_error_nonbool(s) (config_error_nonbool(s), -1)
#endif
extern const char *get_log_output_encoding(void);