From: Paul Eggert <eggert@cs.ucla.edu>
Date: Thu, 18 Nov 2010 00:29:40 +0000 (-0800)
Subject: intprops: new macro INT_BITS_STRLEN_BOUND
X-Git-Tag: v0.1~3611
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=7b3b57be85df017c3ac1cf362dc645a701d8b80b;p=gnulib.git

intprops: new macro INT_BITS_STRLEN_BOUND

* lib/intprops.h (INT_BITS_STRLEN_BOUND): New macro, needed by
ftoastr.h.  This exposes an internal of intprops.h that was formerly
not exposed.  Also, it uses a slightly tighter bound than before;
though this makes no practical difference, we might as well be as
tight as we easily can.
---

diff --git a/ChangeLog b/ChangeLog
index ea0610ab7..0ab0af34b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2010-11-17  Paul Eggert  <eggert@cs.ucla.edu>
 
+	intprops: new macro INT_BITS_STRLEN_BOUND
+	* lib/intprops.h (INT_BITS_STRLEN_BOUND): New macro, needed by
+	ftoastr.h.  This exposes an internal of intprops.h that was formerly
+	not exposed.  Also, it uses a slightly tighter bound than before;
+	though this makes no practical difference, we might as well be as
+	tight as we easily can.
+
 	ftoastr: new module, for lossless conversion of floats to short strings
 	* lib/ftoastr.h, lib/ftoastr.c, lib/dtoastr.c, lib/ldtoastr.c:
 	* modules/ftoastr: New files.
diff --git a/lib/intprops.h b/lib/intprops.h
index 46f4d47d7..0c2787582 100644
--- a/lib/intprops.h
+++ b/lib/intprops.h
@@ -68,13 +68,17 @@
 #  define signed_type_or_expr__(t) 1
 # endif
 
+/* Bound on length of the string representing an unsigned integer
+   value representable in B bits.  log10 (2.0) < 146/485.  The
+   smallest value of B where this bound is not tight is 2621.  */
+# define INT_BITS_STRLEN_BOUND(b) (((b) * 146 + 484) / 485)
+
 /* Bound on length of the string representing an integer type or expression T.
-   Subtract 1 for the sign bit if T is signed; log10 (2.0) < 146/485;
-   add 1 for integer division truncation; add 1 more for a minus sign
-   if needed.  */
+   Subtract 1 for the sign bit if T is signed, and then add 1 more for
+   a minus sign if needed.  */
 # define INT_STRLEN_BOUND(t) \
-  ((sizeof (t) * CHAR_BIT - signed_type_or_expr__ (t)) * 146 / 485 \
-   + signed_type_or_expr__ (t) + 1)
+  (INT_BITS_STRLEN_BOUND (sizeof (t) * CHAR_BIT - signed_type_or_expr__ (t)) \
+   + signed_type_or_expr__ (t))
 
 /* Bound on buffer size needed to represent an integer type or expression T,
    including the terminating null.  */