author | Ramsay Jones <ramsay@ramsay1.demon.co.uk> | |
Wed, 23 Jun 2010 19:40:19 +0000 (20:40 +0100) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Thu, 24 Jun 2010 16:42:12 +0000 (09:42 -0700) | ||
commit | 4e0d7a80186c27f1a008dd10eaea6f25b0e1fe19 | |
tree | 873627558b8f03c583db78abb01185a4aba3dc5a | tree | snapshot |
parent | a4c24549acf16f7917ea3ddd811c6c61afc5689c | commit | diff |
msvc: Fix some "expr evaluates to function" compiler warnings
In particular, the following warning is issued while compiling
notes.c:
notes.c(927) : warning C4550: expression evaluates to a \
function which is missing an argument list
along with identical warnings on lines 928, 1016 and 1017.
In order to suppress the warning, we change the definition of
combine_notes_fn, so that the symbol type is an (explicit)
"pointer to function ...". As a result, several other
declarations need some minor fix-up to take account of the
new typedef.
Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Acked-by: Johan Herland <johan@herland.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
In particular, the following warning is issued while compiling
notes.c:
notes.c(927) : warning C4550: expression evaluates to a \
function which is missing an argument list
along with identical warnings on lines 928, 1016 and 1017.
In order to suppress the warning, we change the definition of
combine_notes_fn, so that the symbol type is an (explicit)
"pointer to function ...". As a result, several other
declarations need some minor fix-up to take account of the
new typedef.
Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Acked-by: Johan Herland <johan@herland.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin.h | diff | blob | history | |
builtin/notes.c | diff | blob | history | |
notes.h | diff | blob | history |