[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] #include limts.h for ULONG_MAX

From: Joe Swatosh <joe.swatosh_at_gmail.com>
Date: Mon, 25 Mar 2013 17:24:39 -0700

Thanks Philip and Bert, perhaps this along with Philip's changes?

[[[
Index: subversion/include/svn_types.h
===================================================================
--- subversion/include/svn_types.h (revision 1460925)
+++ subversion/include/svn_types.h (working copy)
@@ -1241,7 +1241,7 @@
  *
  * @since New in 1.7.
  */
-#define SVN_LINENUM_MAX_VALUE ULONG_MAX
+#define SVN_LINENUM_MAX_VALUE APR_UINT64_MAX

]]]

On Mon, Mar 25, 2013 at 4:54 AM, Philip Martin
<philip.martin_at_wandisco.com> wrote:
> In r1460628 I changed the code to use svn_cstring_strtoui64 and so avoid
> ULONG_MAX.
>
> Bert Huijben <bert_at_qqmail.nl> writes:
>
>> 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).
>
> --
> Certified & Supported Apache Subversion Downloads:
> http://www.wandisco.com/subversion/download
Received on 2013-03-26 01:25:11 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.