Changeset 46577995 in mainline for uspace/app/bdsh/cmds/builtins/cd/cd.c

Timestamp:

2018-01-04T20:50:52Z (7 years ago)

Author:

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

Children:

e211ea04

Parents:

facacc71

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:47:53)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:50:52)

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.

After this commit, HelenOS is free of code that mixes error codes with non-error
values on the assumption that error codes are negative.

File:

• uspace/app/bdsh/cmds/builtins/cd/cd.c (modified) (3 diffs)

uspace/app/bdsh/cmds/builtins/cd/cd.c

@@ -51 +51 @@
 static bool previous_directory_set = false;
 
-static int chdir_and_remember(const char *new_dir)
+static errno_t chdir_and_remember(const char *new_dir)
 {
 
-        int rc = vfs_cwd_get(previous_directory_tmp, PATH_MAX);
+        errno_t rc = vfs_cwd_get(previous_directory_tmp, PATH_MAX);
         previous_directory_valid = (rc == EOK);
         previous_directory_set = true;
@@ -86 +86 @@
 {
         int argc;
-        int rc = EOK;
+        errno_t rc = EOK;
 
         argc = cli_count_args(argv);
@@ -145 +145 @@
                 return CMD_SUCCESS;
         } else {
-                switch (rc) {
-                case ENOMEM:
+                switch ((case_errno_t) rc) {
+                case (case_errno_t) ENOMEM:
                         cli_error(CL_EFAIL, "Destination path too long");
                         break;
-                case ENOENT:
+                case (case_errno_t) ENOENT:
                         cli_error(CL_ENOENT, "Invalid directory `%s'", target_directory);
                         break;