Changes in kernel/generic/src/console/kconsole.c [feeac0d:aca4a04] in mainline

File:

• kernel/generic/src/console/kconsole.c (modified) (11 diffs)

kernel/generic/src/console/kconsole.c

@@ -53 +53 @@
 #include <func.h>
 #include <str.h>
+#include <macros.h>
 #include <sysinfo/sysinfo.h>
 #include <ddi/device.h>
@@ -118 +119 @@
          * Make sure the command is not already listed.
          */
-        list_foreach(cmd_list, link, cmd_info_t, hlp) {
+        list_foreach(cmd_list, cur) {
+                cmd_info_t *hlp = list_get_instance(cur, cmd_info_t, link);
+                
                 if (hlp == cmd) {
                         /* The command is already there. */
@@ -199 +202 @@
  *
  */
-NO_TRACE static int cmdtab_compl(char *input, size_t size, indev_t *indev)
+NO_TRACE static int cmdtab_compl(char *input, size_t size, indev_t * indev)
 {
         const char *name = input;
         
         size_t found = 0;
-        
-        /*
-         * Maximum Match Length: Length of longest matching common
-         * substring in case more than one match is found.
-         */
+        /* Maximum Match Length : Length of longest matching common substring in
+           case more than one match is found */
         size_t max_match_len = size;
         size_t max_match_len_tmp = size;
@@ -229 +229 @@
         }
         
-        /*
-         * If the number of possible completions is more than MAX_TAB_HINTS,
-         * ask the user whether to display them or not.
-         */
+        /* If possible completions are more than MAX_TAB_HINTS, ask user whether to display them or not. */
         if (found > MAX_TAB_HINTS) {
                 printf("\n");
-                continue_showing_hints =
-                    console_prompt_display_all_hints(indev, found);
+                continue_showing_hints = console_prompt_display_all_hints(indev, found);
         }
         
@@ -244 +240 @@
                 while (cmdtab_search_one(name, &pos)) {
                         cmd_info_t *hlp = list_get_instance(pos, cmd_info_t, link);
-                        
+
                         if (continue_showing_hints) {
                                 printf("%s (%s)\n", hlp->name, hlp->description);
                                 --hints_to_show;
                                 ++total_hints_shown;
-                                
-                                if ((hints_to_show == 0) && (total_hints_shown != found)) {
-                                        /* Ask user to continue */
-                                        continue_showing_hints =
-                                            console_prompt_more_hints(indev, &hints_to_show);
+
+                                if (hints_to_show == 0 && total_hints_shown != found) { /* Time to ask user to continue */
+                                        continue_showing_hints = console_prompt_more_hints(indev, &hints_to_show);
                                 }
                         }
-                        
+
                         pos = pos->next;
-                        
-                        for (max_match_len_tmp = 0;
-                            (output[max_match_len_tmp] ==
-                            hlp->name[input_len + max_match_len_tmp]) &&
-                            (max_match_len_tmp < max_match_len); ++max_match_len_tmp);
-                        
+                        for(max_match_len_tmp = 0; output[max_match_len_tmp] == hlp->name[input_len + max_match_len_tmp]
+                                        && max_match_len_tmp < max_match_len; ++max_match_len_tmp);
                         max_match_len = max_match_len_tmp;
                 }
-                
-                /* Keep only the characters common in all completions */
+                /* keep only the characters common in all completions */
                 output[max_match_len] = 0;
         }
@@ -321 +310 @@
                                 continue;
                         
-                        /*
-                         * Find the beginning of the word
-                         * and copy it to tmp
-                         */
+                        /* Find the beginning of the word
+                           and copy it to tmp */
                         size_t beg;
                         for (beg = position - 1; (beg > 0) && (!isspace(current[beg]));
@@ -346 +333 @@
                                 continue;
 
-                        /*
-                         * We have hints, possibly many. In case of more than one hint,
-                         * tmp will contain the common prefix.
-                         */
+                        /* We have hints, may be many. In case of more than one hint,
+                           tmp will contain the common prefix. */
                         size_t off = 0;
                         size_t i = 0;
@@ -355 +340 @@
                                 if (!wstr_linsert(current, ch, position + i, MAX_CMDLINE))
                                         break;
-                                
                                 i++;
                         }
@@ -521 +505 @@
                 /* It's a number - convert it */
                 uint64_t value;
-                char *end;
-                int rc = str_uint64_t(text, &end, 0, false, &value);
-                if (end != text + len)
-                        rc = EINVAL;
+                int rc = str_uint64_t(text, NULL, 0, true, &value);
                 switch (rc) {
                 case EINVAL:
-                        printf("Invalid number '%s'.\n", text);
+                        printf("Invalid number.\n");
                         return false;
                 case EOVERFLOW:
-                        printf("Integer overflow in '%s'.\n", text);
+                        printf("Integer overflow.\n");
                         return false;
                 case EOK:
@@ -538 +519 @@
                         break;
                 default:
-                        printf("Unknown error parsing '%s'.\n", text);
+                        printf("Unknown error.\n");
                         return false;
                 }
@@ -610 +591 @@
         cmd_info_t *cmd = NULL;
         
-        list_foreach(cmd_list, link, cmd_info_t, hlp) {
+        list_foreach(cmd_list, cur) {
+                cmd_info_t *hlp = list_get_instance(cur, cmd_info_t, link);
                 spinlock_lock(&hlp->lock);