[cvsnt-dev] cvsnt does not compile with upcoming GCC4.3
Tony Hoyle
tony.hoyle at march-hare.com
Mon Apr 2 14:28:33 BST 2007
Andreas Tscharner wrote:
> +#include <climits>
If it's a c header that should be limits.h - too many portability issues
with trying to use the c++ wrappers (not all implementations have them,
some of them - including a version of gcc IIRC - put the resulting
include into a namespace, etc.).
OTOH there's no reason at all why that should be there - Neither
StringDiff or DiffBase have any conceivable use for that file.
> #ifdef __cplusplus
>
> +#include <cstdlib>
> +
cvsdelta.cpp already includes stdilb.h. Adding an extra include here
serves no purpose.
I'd rather see the error output to see what is wrong.
Tony
More information about the cvsnt-dev
mailing list