Changes in / [e435537:9904eb90] in mainline


Ignore:
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • kernel/generic/src/console/kconsole.c

    re435537 r9904eb90  
    202202 *
    203203 */
    204 NO_TRACE static int cmdtab_compl(char *input, size_t size, indev_t *indev)
     204NO_TRACE static int cmdtab_compl(char *input, size_t size, indev_t * indev)
    205205{
    206206        const char *name = input;
    207207       
    208208        size_t found = 0;
    209        
    210         /*
    211          * Maximum Match Length: Length of longest matching common
    212          * substring in case more than one match is found.
    213          */
     209        /* Maximum Match Length : Length of longest matching common substring in
     210           case more than one match is found */
    214211        size_t max_match_len = size;
    215212        size_t max_match_len_tmp = size;
    … …  
    232229        }
    233230       
    234         /*
    235          * If the number of possible completions is more than MAX_TAB_HINTS,
    236          * ask the user whether to display them or not.
    237          */
     231        /* If possible completions are more than MAX_TAB_HINTS, ask user whether to display them or not. */
    238232        if (found > MAX_TAB_HINTS) {
    239233                printf("\n");
    240                 continue_showing_hints =
    241                     console_prompt_display_all_hints(indev, found);
     234                continue_showing_hints = console_prompt_display_all_hints(indev, found);
    242235        }
    243236       
    … …  
    247240                while (cmdtab_search_one(name, &pos)) {
    248241                        cmd_info_t *hlp = list_get_instance(pos, cmd_info_t, link);
    249                        
     242
    250243                        if (continue_showing_hints) {
    251244                                printf("%s (%s)\n", hlp->name, hlp->description);
    252245                                --hints_to_show;
    253246                                ++total_hints_shown;
    254                                
    255                                 if ((hints_to_show == 0) && (total_hints_shown != found)) {
    256                                         /* Ask user to continue */
    257                                         continue_showing_hints =
    258                                             console_prompt_more_hints(indev, &hints_to_show);
     247
     248                                if (hints_to_show == 0 && total_hints_shown != found) { /* Time to ask user to continue */
     249                                        continue_showing_hints = console_prompt_more_hints(indev, &hints_to_show);
    259250                                }
    260251                        }
    261                        
     252
    262253                        pos = pos->next;
    263                        
    264                         for (max_match_len_tmp = 0;
    265                             (output[max_match_len_tmp] ==
    266                             hlp->name[input_len + max_match_len_tmp]) &&
    267                             (max_match_len_tmp < max_match_len); ++max_match_len_tmp);
    268                        
     254                        for(max_match_len_tmp = 0; output[max_match_len_tmp] == hlp->name[input_len + max_match_len_tmp]
     255                                        && max_match_len_tmp < max_match_len; ++max_match_len_tmp);
    269256                        max_match_len = max_match_len_tmp;
    270257                }
    271                
    272                 /* Keep only the characters common in all completions */
     258                /* keep only the characters common in all completions */
    273259                output[max_match_len] = 0;
    274260        }
    … …  
    324310                                continue;
    325311                       
    326                         /*
    327                          * Find the beginning of the word
    328                          * and copy it to tmp
    329                          */
     312                        /* Find the beginning of the word
     313                           and copy it to tmp */
    330314                        size_t beg;
    331315                        for (beg = position - 1; (beg > 0) && (!isspace(current[beg]));
    … …  
    349333                                continue;
    350334
    351                         /*
    352                          * We have hints, possibly many. In case of more than one hint,
    353                          * tmp will contain the common prefix.
    354                          */
     335                        /* We have hints, may be many. In case of more than one hint,
     336                           tmp will contain the common prefix. */
    355337                        size_t off = 0;
    356338                        size_t i = 0;
    … …  
    358340                                if (!wstr_linsert(current, ch, position + i, MAX_CMDLINE))
    359341                                        break;
    360                                
    361342                                i++;
    362343                        }

  • kernel/generic/src/console/prompt.c

    re435537 r9904eb90  
    4343 * @param indev Where to read characters from.
    4444 * @param hints Number of hints that would be displayed.
    45  *
    4645 * @return Whether to print all hints.
    47  *
    4846 */
    4947bool console_prompt_display_all_hints(indev_t *indev, size_t hints)
    … …  
    5149        ASSERT(indev);
    5250        ASSERT(hints > 0);
    53        
    54         printf("Display all %zu possibilities? (y or n) ", hints);
    55        
     51
     52        printf("Display all %zu possibilities? (y or n)", hints);
     53
    5654        while (true) {
    5755                wchar_t answer = indev_pop_character(indev);
    58                
    59                 if ((answer == 'y') || (answer == 'Y')) {
    60                         printf("y");
     56
     57                if (answer == 'y' || answer == 'Y') {
     58                        printf(" y");
    6159                        return true;
    6260                }
    63                
    64                 if ((answer == 'n') || (answer == 'N')) {
    65                         printf("n");
     61
     62                if (answer == 'n' || answer == 'N') {
     63                        printf(" n");
    6664                        return false;
    6765                }
    … …  
    7371 * When the function returns false, @p display_hints is set to zero.
    7472 *
    75  * @param[in]  indev        Where to read characters from.
     73 * @param[in] indev Where to read characters from.
    7674 * @param[out] display_hints How many hints to display.
    77  *
    7875 * @return Whether to display more hints.
    79  *
    8076 */
    8177bool console_prompt_more_hints(indev_t *indev, size_t *display_hints)
    … …  
    8379        ASSERT(indev);
    8480        ASSERT(display_hints != NULL);
    85        
     81
    8682        printf("--More--");
    8783        while (true) {
    8884                wchar_t continue_showing_hints = indev_pop_character(indev);
    8985                /* Display a full page again? */
    90                 if ((continue_showing_hints == 'y') ||
    91                     (continue_showing_hints == 'Y') ||
    92                     (continue_showing_hints == ' ')) {
     86                if (continue_showing_hints == 'y'
     87                    || continue_showing_hints == 'Y'
     88                    || continue_showing_hints == ' ') {
    9389                        *display_hints = MAX_TAB_HINTS - 1;
    9490                        break;
    9591                }
    96                
     92
    9793                /* Stop displaying hints? */
    98                 if ((continue_showing_hints == 'n') ||
    99                     (continue_showing_hints == 'N') ||
    100                     (continue_showing_hints == 'q') ||
    101                     (continue_showing_hints == 'Q')) {
     94                if (continue_showing_hints == 'n'
     95                    || continue_showing_hints == 'N'
     96                    || continue_showing_hints == 'q'
     97                    || continue_showing_hints == 'Q') {
    10298                        *display_hints = 0;
    10399                        break;
    104100                }
    105                
     101
    106102                /* Show one more hint? */
    107103                if (continue_showing_hints == '\n') {
    … …  
    110106                }
    111107        }
    112        
     108
    113109        /* Delete the --More-- option */
    114110        printf("\r         \r");
    115        
     111
    116112        return *display_hints > 0;
    117113}

  • kernel/generic/src/debug/symtab.c

    re435537 r9904eb90  
    210210 *
    211211 */
    212 int symtab_compl(char *input, size_t size, indev_t *indev)
     212int symtab_compl(char *input, size_t size, indev_t * indev)
    213213{
    214214#ifdef CONFIG_SYMTAB
    … …  
    227227        const char *hint;
    228228        char output[MAX_SYMBOL_NAME];
    229        
    230         /*
    231          * Maximum Match Length: Length of longest matching common substring in
    232          * case more than one match is found.
    233          */
     229        /* Maximum Match Length : Length of longest matching common substring in
     230           case more than one match is found */
    234231        size_t max_match_len = size;
    235232        size_t max_match_len_tmp = size;
    … …  
    241238       
    242239        output[0] = 0;
    243        
    244         while ((hint = symtab_search_one(name, &pos)))
    245                 pos++;
    246        
     240
     241        while ((hint = symtab_search_one(name, &pos))) {
     242                ++pos;
     243        }
     244
    247245        pos = 0;
    248246       
    … …  
    255253        }
    256254       
    257         /*
    258          * If the number of possible completions is more than MAX_TAB_HINTS,
    259          * ask the user whether to display them or not.
    260          */
     255        /* If possible completions are more than MAX_TAB_HINTS, ask user whether to display them or not. */
    261256        if (found > MAX_TAB_HINTS) {
    262257                printf("\n");
    263                 continue_showing_hints =
    264                     console_prompt_display_all_hints(indev, found);
     258                continue_showing_hints = console_prompt_display_all_hints(indev, found);
    265259        }
    266260       
    … …  
    271265                        sym_name = symbol_table[pos].symbol_name;
    272266                        pos++;
    273                        
    274                         if (continue_showing_hints) {
    275                                 /* We are still showing hints */
     267
     268                        if (continue_showing_hints) { /* We are still showing hints */
    276269                                printf("%s\n", sym_name);
    277270                                --hints_to_show;
    278271                                ++total_hints_shown;
    279                                
    280                                 if ((hints_to_show == 0) && (total_hints_shown != found)) {
    281                                         /* Ask the user to continue */
    282                                         continue_showing_hints =
    283                                             console_prompt_more_hints(indev, &hints_to_show);
     272
     273                                if (hints_to_show == 0 && total_hints_shown != found) { /* Time to ask user to continue */
     274                                        continue_showing_hints = console_prompt_more_hints(indev, &hints_to_show);
    284275                                }
    285276                        }
    286                        
    287                         for (max_match_len_tmp = 0;
    288                             (output[max_match_len_tmp] ==
    289                             sym_name[input_len + max_match_len_tmp]) &&
    290                             (max_match_len_tmp < max_match_len); ++max_match_len_tmp);
    291                        
     277
     278                        for(max_match_len_tmp = 0; output[max_match_len_tmp] == sym_name[input_len + max_match_len_tmp]
     279                                        && max_match_len_tmp < max_match_len; ++max_match_len_tmp);
    292280                        max_match_len = max_match_len_tmp;
    293281                }
    294                
    295                 /* Keep only the characters common in all completions */
     282                /* keep only the characters common in all completions */
    296283                output[max_match_len] = 0;
    297284        }

  • uspace/app/bdsh/cmds/modules/cat/cat.c

    re435537 r9904eb90  
    176176
    177177        bool reading_stdin = dash_represents_stdin && (str_cmp(fname, "-") == 0);
    178        
     178
    179179        if (reading_stdin) {
    180180                fd = fileno(stdin);
    181181                /* Allow storing the whole UTF-8 character. */
    182182                blen = STR_BOUNDS(1);
    183         } else
     183        } else {
    184184                fd = open(fname, O_RDONLY);
    185        
     185        }
    186186        if (fd < 0) {
    187187                printf("Unable to open %s\n", fname);
    … …  
    222222                        bytes_to_read = 1;
    223223                } else {
    224                         if ((length != CAT_FULL_FILE) &&
    225                             (length - (off64_t)count <= (off64_t)(blen - copied_bytes))) {
     224                        if ((length != CAT_FULL_FILE)
     225                            && (length - (off64_t)count <= (off64_t)(blen - copied_bytes))) {
    226226                                bytes_to_read = (size_t) (length - count);
    227227                        } else {
    … …  
    229229                        }
    230230                }
    231                
    232231                bytes = read(fd, buff + copied_bytes, bytes_to_read);
    233232                bytes += copied_bytes;
    … …  
    262261                        reads++;
    263262                }
    264                
    265                 if (reading_stdin)
     263
     264                if (reading_stdin) {
    266265                        fflush(stdout);
     266                }
    267267        } while (bytes > 0 && !should_quit && (count < length || length == CAT_FULL_FILE));
    268268

  • uspace/srv/hid/console/console.c

    re435537 r9904eb90  
    617617       
    618618        size_t pos = 0;
    619        
     619
    620620        /*
    621621         * Read input from keyboard and copy it to the buffer.
    … …  
    628628                        buf[pos] = cons->char_remains[0];
    629629                        pos++;
    630                        
    631630                        /* Unshift the array. */
    632                         for (size_t i = 1; i < cons->char_remains_len; i++)
     631                        for (size_t i = 1; i < cons->char_remains_len; i++) {
    633632                                cons->char_remains[i - 1] = cons->char_remains[i];
    634                        
     633                        }
    635634                        cons->char_remains_len--;
    636635                }
    637                
    638636                /* Still not enough? Then get another key from the queue. */
    639637                if (pos < size) {
    640638                        link_t *link = prodcons_consume(&cons->input_pc);
    641639                        kbd_event_t *event = list_get_instance(link, kbd_event_t, link);
    642                        
     640
    643641                        /* Accept key presses of printable chars only. */
    644642                        if ((event->type == KEY_PRESS) && (event->c != 0)) {
    … …  
    647645                                cons->char_remains_len = str_size(cons->char_remains);
    648646                        }
    649                        
     647
    650648                        free(event);
    651649                }
Note: See TracChangeset for help on using the changeset viewer.