[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


Community technical support mailing list was retired 2010 and replaced with a professional technical support team. For assistance please contact: Pre-sales Technical support via email to sales@march-hare.com.


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