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

Timestamp:

2019-02-11T14:08:52Z (6 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.

File:

• uspace/lib/c/generic/imath.c (modified) (2 diffs)

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;
 }