Changeset b7fd2a0 in mainline for uspace/app/sbi/src/input.c


Ignore:
Timestamp:
2018-01-13T03:10:29Z (8 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
a53ed3a
Parents:
36f0738
Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

Although this is a massive commit, it is a simple text replacement, and thus
is very easy to verify. Simply do the following:

`
git checkout <this commit's hash>
git reset HEAD
git add .
tools/srepl '\berrno_t\b' int
git add .
tools/srepl '\bsys_errno_t\b' sysarg_t
git reset
git diff
`

While this doesn't ensure that the replacements are correct, it does ensure
that the commit doesn't do anything except those replacements. Since errno_t
is typedef'd to int in the usual case (and sys_errno_t to sysarg_t), even if
incorrect, this commit cannot change behavior.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • uspace/app/sbi/src/input.c

    r36f0738 rb7fd2a0  
    4545#define INPUT_BUFFER_SIZE 256
    4646
    47 static int input_init_file(input_t *input, const char *fname);
     47static errno_t input_init_file(input_t *input, const char *fname);
    4848static void input_init_interactive(input_t *input);
    4949static void input_init_string(input_t *input, const char *str);
     
    5757 *                      ENOENT when opening file fails.
    5858 */
    59 int input_new_file(input_t **input, const char *fname)
     59errno_t input_new_file(input_t **input, const char *fname)
    6060{
    6161        *input = malloc(sizeof(input_t));
     
    7171 * @return              EOK on success, ENOMEM when allocation fails.
    7272 */
    73 int input_new_interactive(input_t **input)
     73errno_t input_new_interactive(input_t **input)
    7474{
    7575        *input = malloc(sizeof(input_t));
     
    8787 * @return              EOK on success, ENOMEM when allocation fails.
    8888 */
    89 int input_new_string(input_t **input, const char *str)
     89errno_t input_new_string(input_t **input, const char *str)
    9090{
    9191        *input = malloc(sizeof(input_t));
     
    104104 * @return              EOK on success, ENOENT when opening file fails.
    105105*/
    106 static int input_init_file(input_t *input, const char *fname)
     106static errno_t input_init_file(input_t *input, const char *fname)
    107107{
    108108        FILE *f;
     
    174174 * @return              EOK on success, EIO on failure.
    175175 */
    176 int input_get_line(input_t *input, char **line)
     176errno_t input_get_line(input_t *input, char **line)
    177177{
    178178        const char *prompt;
Note: See TracChangeset for help on using the changeset viewer.