Changeset d09f84e6 in mainline for kernel/generic/src/lib/string.c

Timestamp:

2009-04-02T19:59:22Z (15 years ago)

Author:

Jiri Svoboda <jirik.svoboda@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

82bb9c1

Parents:

06b785f

Message:

Return error code from chr_encode() rather than boolean.

File:

• kernel/generic/src/lib/string.c (modified) (5 diffs)

kernel/generic/src/lib/string.c

@@ -41 +41 @@
 #include <arch/asm.h>
 #include <arch.h>
+#include <errno.h>
 #include <console/kconsole.h>
 
@@ -141 +142 @@
  * @param sz            Size of the output buffer.
  *
- * @return True if the character was encoded successfully or false if there
- *         was not enough space in the output buffer or the character code
- *         was invalid.
- */
-bool chr_encode(const wchar_t ch, char *str, size_t *offset, size_t sz)
+ * @return EOK if the character was encoded successfully, EOVERFLOW if there
+ *         was not enough space in the output buffer or EINVAL if the character
+ *         code was invalid.
+ */
+int chr_encode(const wchar_t ch, char *str, size_t *offset, size_t sz)
 {
         uint32_t cc;            /* Unsigned version of ch. */
@@ -154 +155 @@
 
         if (*offset >= sz)
-                return false;
+                return EOVERFLOW;
 
         if (ch < 0)
-                return false;
+                return EINVAL;
 
         /* Bit operations should only be done on unsigned numbers. */
@@ -177 +178 @@
         } else {
                 /* Codes longer than 21 bits are not supported. */
-                return false;
+                return EINVAL;
         }
 
         /* Check for available space in buffer. */
         if (*offset + cbytes >= sz)
-                return false;
+                return EOVERFLOW;
 
         /* Encode continuation bytes. */
@@ -196 +197 @@
         *offset += (1 + cbytes);
         
-        return true;
+        return EOK;
 }