diff options
author | Jeff King <peff@peff.net> | 2014-05-06 11:17:50 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-05-06 15:30:40 -0700 |
commit | ff0a80af724e81dbad6a269847523e39c2e7e479 (patch) | |
tree | f2af81cd7cde8d61559dadfbaec8693b47532181 /test-line-buffer.c | |
parent | 87fe5df3653cf20b6bf9854bea42e4016c7d4688 (diff) | |
download | git-ff0a80af724e81dbad6a269847523e39c2e7e479.tar.gz |
let clang use the constant-return error() macrojk/squelch-compiler-warning-from-funny-error-macro
Commit e208f9c converted error() into a macro to make its
constant return value more apparent to calling code. Commit
5ded807 prevents us using this macro with clang, since
clang's -Wunused-value is smart enough to realize that the
constant "-1" is useless in some contexts.
However, since the last commit puts the constant behind an
inline function call, this is enough to prevent the
-Wunused-value warning on both modern gcc and clang. So we
can now re-enable the macro when compiling with clang.
Tested with clang 3.3, 3.4, and 3.5.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'test-line-buffer.c')
0 files changed, 0 insertions, 0 deletions