msvc: test-svn-fe: Fix linker "unresolved external" error
commitd0f9dbb9e29b8dc289ae3b8028586a99fdb0d194
authorRamsay Jones <ramsay@ramsay1.demon.co.uk>
Thu, 31 Jan 2013 18:32:55 +0000 (31 18:32 +0000)
committerJunio C Hamano <gitster@pobox.com>
Mon, 25 Feb 2013 20:40:34 +0000 (25 12:40 -0800)
tree495515c22e0ed9901bd3666858d04cd94d9766d2
parent93e38ed0c5a632d7b785357a309ac83e4df066fa
msvc: test-svn-fe: Fix linker "unresolved external" error

In particular, while linking test-svn-fe.exe, the linker complains
that the external symbol _strtoull is unresolved. A call to this
function was added in commit ddcc8c5b ("vcs-svn: skeleton of an svn
delta parser", 25-12-2010).

The NO_STRTOULL build variable attempts to provide support to old
systems which can't even declare 'unsigned long long' variables,
let alone provide the strtoll() or strtoull() functions. Setting
this build variable does not provide an implementation of these
functions. Rather, it simply allows the compat implementations
of strto{i,u}max() to use strtol() and strtoul() instead.

In order to fix the linker error on systems with NO_STRTOULL set,
currently MSVC and OSF1, we can substitute a call to strtoumax().

However, we can easily provide support for the strtoull() and
strtoll() functions on MSVC, since they are essentially already
available as _strtoui64() and _strtoi64(). This allows us to
remove NO_STRTOULL for MSVC.

Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Tested-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
compat/msvc.h
config.mak.uname
test-svn-fe.c