Changeset e3272101 in mainline for uspace/lib/c/generic

Timestamp:

2019-02-11T14:08:52Z (7 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

d5a89a3

Parents:

aa57bf7

Message:

make ilog10_u64() return an errno_t

The function ilog10_u64() used to return 0
for the value 0. Which is not correct. Either
NaN or -Infinity are correct, but not 0, since
it would be ambiguous with log(1). To ensure this
case the function ilog10_u64() has been changed
to return a errno_t indicating a failure.

Location:

uspace/lib/c/generic

Files:

• cap.c (modified) (1 diff)
• imath.c (modified) (2 diffs)

uspace/lib/c/generic/cap.c

@@ -160 +160 @@
 
         /* Round the number so that we have at most @c scap_max_sdig significant digits */
-        sdig = 1 + ilog10_u64(cap->m); /* number of significant digits */
+        rc = ilog10_u64(cap->m, &sdig); /* number of significant digits */
+        sdig += 1;
+        assert(rc == EOK);
         if (sdig > scap_max_sdig) {
                 /* Number of digits to remove */

uspace/lib/c/generic/imath.c

@@ -81 +81 @@
  *
  * @param v Value to compute logarithm from
+ * @param res Place to store result
+ * @return EOK on success
  * @return Logarithm value
  */
-unsigned ilog10_u64(uint64_t v)
+errno_t ilog10_u64(uint64_t v, unsigned *res)
 {
+        if (v == 0)
+                return ERANGE;
+
         unsigned b;
         unsigned e;
@@ -117 +122 @@
         }
 
-        return r;
+        *res = r;
+        return EOK;
 }