Isn't there an apr variant of this macro we can/should use?

Bert From: Joe Swatosh
Sent: 25-3-2013 01:01
To: SVN Dev
Subject: [PATCH] #include limts.h for ULONG_MAX
While I found this issue using my very old compiler, it seems reasonable since
that is where ULONG_MAX has been defined for a long time. (Since clearly no one
else is running into this).

[[[
For those of us with somewhat deficient compilers that don't provide ULONG_MAX
unless <limits.h> is explicitly included.

 * subversion/include/svn_types.h
   (includes): Add include of limits.h
]]]

Index: subversion/include/svn_types.h
===================================================================
--- subversion/include/svn_types.h (revision 1460195)
+++ subversion/include/svn_types.h (working copy)
@@ -29,6 +29,7 @@

 /* ### this should go away, but it causes too much breakage right now */
 #include <stdlib.h>
+#include <limits.h> /* for ULONG_MAX */

 #include <apr.h>         /* for apr_size_t, apr_int64_t, ... */
 #include <apr_errno.h>   /* for apr_status_t */

Reply via email to