Changeset b7fd2a0 in mainline for uspace/app/edit/edit.c

Timestamp:

2018-01-13T03:10:29Z (7 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
<sup>git add .
tools/srepl '\berrno_t\b' int
git add .
tools/srepl '\bsys_errno_t\b' sysarg_t
git reset
git diff
`</sup>

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:

• uspace/app/edit/edit.c (modified) (11 diffs)

uspace/app/edit/edit.c

@@ -132 +132 @@
 static void pos_handle(pos_event_t *ev);
 
-static int file_save(char const *fname);
+static errno_t file_save(char const *fname);
 static void file_save_as(void);
-static int file_insert(char *fname);
-static int file_save_range(char const *fname, spt_t const *spos,
+static errno_t file_insert(char *fname);
+static errno_t file_save_range(char const *fname, spt_t const *spos,
     spt_t const *epos);
 static char *range_get_str(spt_t const *spos, spt_t const *epos);
@@ -191 +191 @@
         cons_event_t ev;
         bool new_file;
-        int rc;
+        errno_t rc;
 
         con = console_init(stdin, stdout);
@@ -578 +578 @@
 
 /** Save the document. */
-static int file_save(char const *fname)
+static errno_t file_save(char const *fname)
 {
         spt_t sp, ep;
-        int rc;
+        errno_t rc;
 
         status_display("Saving...");
@@ -616 +616 @@
         }
 
-        int rc = file_save(fname);
+        errno_t rc = file_save(fname);
         if (rc != EOK)
                 return;
@@ -697 +697 @@
  * of the caret.
  */
-static int file_insert(char *fname)
+static errno_t file_insert(char *fname)
 {
         FILE *f;
@@ -735 +735 @@
 
 /** Save a range of text into a file. */
-static int file_save_range(char const *fname, spt_t const *spos,
+static errno_t file_save_range(char const *fname, spt_t const *spos,
     spt_t const *epos)
 {
@@ -1283 +1283 @@
 
 /* Search operations */
-static int search_spt_producer(void *data, wchar_t *ret)
+static errno_t search_spt_producer(void *data, wchar_t *ret)
 {
         assert(data != NULL);
@@ -1292 +1292 @@
 }
 
-static int search_spt_reverse_producer(void *data, wchar_t *ret)
+static errno_t search_spt_reverse_producer(void *data, wchar_t *ret)
 {
         assert(data != NULL);
@@ -1301 +1301 @@
 }
 
-static int search_spt_mark(void *data, void **mark)
+static errno_t search_spt_mark(void *data, void **mark)
 {
         assert(data != NULL);
@@ -1397 +1397 @@
         
         match_t match;
-        int rc = search_next_match(search, &match);
+        errno_t rc = search_next_match(search, &match);
         if (rc != EOK) {
                 status_display("Failed searching.");
@@ -1515 +1515 @@
         size_t off;
         wchar_t c;
-        int rc;
+        errno_t rc;
 
         rc = clipboard_get_str(&str);