Changeset 08e103d4 in mainline for kernel/genarch

Timestamp:

2019-02-05T18:26:05Z (7 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Children:

1d2f85e

Parents:

d066259

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-05 16:16:55)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-05 18:26:05)

Message:

Use clearer naming for string length functions

This and the following commit change the names of functions, as well as
their documentation, to use unambiguous terms "bytes" and "code points"
instead of ambiguous terms "size", "length", and "characters".

Location:

kernel/genarch/src

Files:

• multiboot/multiboot.c (modified) (2 diffs)
• ofw/ofw_tree.c (modified) (4 diffs)

kernel/genarch/src/multiboot/multiboot.c

@@ -51 +51 @@
         const char *end = str_chr(cmd_line, ' ');
         if (end == NULL)
-                end = cmd_line + str_size(cmd_line);
+                end = cmd_line + str_bytes(cmd_line);
 
         /*
@@ -88 +88 @@
         }
 
-        const char *end = cmd_line + str_size(cmd_line);
+        const char *end = cmd_line + str_bytes(cmd_line);
 
         /* Skip the space(s). */

kernel/genarch/src/ofw/ofw_tree.c

@@ -250 +250 @@
         size_t j;
 
-        for (size_t i = 1; (i < str_size(path)) && (node); i = j + 1) {
+        for (size_t i = 1; (i < str_bytes(path)) && (node); i = j + 1) {
                 j = i;
-                while (j < str_size(path) && path[j] != '/')
+                while (j < str_bytes(path) && path[j] != '/')
                         j++;
 
@@ -342 +342 @@
         *size = 0;
         for (size_t i = 0; i < node->properties; i++)
-                *size += str_size(node->property[i].name) + 1 +
+                *size += str_bytes(node->property[i].name) + 1 +
                     sizeof(node->property[i].size) + node->property[i].size;
 
@@ -359 +359 @@
                 /* Property name */
                 str_cpy(dump + pos, *size - pos, node->property[i].name);
-                pos += str_size(node->property[i].name) + 1;
+                pos += str_bytes(node->property[i].name) + 1;
 
                 /* Value size */
@@ -393 +393 @@
                 if ((cur->parent) && (path))
                         snprintf(cur_path, PATH_MAX_LEN, "%s.%s", path, cur->da_name);
-                else if (!str_size(cur->da_name))
+                else if (!str_bytes(cur->da_name))
                         snprintf(cur_path, PATH_MAX_LEN, "firmware.ofw");
                 else