Changeset b7fd2a0 in mainline for uspace/dist/src/c/demos/edit

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
<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.

Location:

uspace/dist/src/c/demos/edit

Files:

• edit.c (modified) (5 diffs)
• search.c (modified) (1 diff)
• search.h (modified) (2 diffs)
• sheet.c (modified) (3 diffs)
• sheet.h (modified) (1 diff)

uspace/dist/src/c/demos/edit/edit.c

@@ -190 +190 @@
         cons_event_t ev;
         bool new_file;
-        int rc;
+        errno_t rc;
 
         con = console_init(stdin, stdout);
@@ -580 +580 @@
 {
         spt_t sp, ep;
-        int rc;
+        errno_t rc;
 
         status_display("Saving...");
@@ -615 +615 @@
         }
 
-        int rc = file_save(fname);
+        errno_t rc = file_save(fname);
         if (rc != EOK)
                 return;
@@ -1330 +1330 @@
         
         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.");
@@ -1448 +1448 @@
         size_t off;
         wchar_t c;
-        int rc;
+        errno_t rc;
 
         rc = clipboard_get_str(&str);

uspace/dist/src/c/demos/edit/search.c

@@ -104 +104 @@
 }
 
-int search_next_match(search_t *s, match_t *match)
+errno_t search_next_match(search_t *s, match_t *match)
 {
         search_equals_fn eq = s->ops.equals;
         
         wchar_t cur_char;
-        int rc = EOK;
+        errno_t rc = EOK;
         while ((rc = s->ops.producer(s->client_data, &cur_char)) == EOK && cur_char > 0) {
                 /* Deal with mismatches */

uspace/dist/src/c/demos/edit/search.h

@@ -42 +42 @@
 typedef struct search search_t;
 typedef bool (*search_equals_fn)(const wchar_t, const wchar_t);
-typedef int (*search_producer_fn)(void *, wchar_t *);
-typedef int (*search_mark_fn)(void *, void **);
+typedef errno_t (*search_producer_fn)(void *, wchar_t *);
+typedef errno_t (*search_mark_fn)(void *, void **);
 typedef void (*search_mark_free_fn)(void *);
 
@@ -60 +60 @@
 extern bool char_exact_equals(const wchar_t, const wchar_t);
 extern search_t *search_init(const char *, void *, search_ops_t, bool);
-extern int search_next_match(search_t *, match_t *);
+extern errno_t search_next_match(search_t *, match_t *);
 extern void search_fini(search_t *);
 

uspace/dist/src/c/demos/edit/sheet.c

@@ -67 +67 @@
 
 /** Initialize an empty sheet. */
-int sheet_create(sheet_t **rsh)
+errno_t sheet_create(sheet_t **rsh)
 {
         sheet_t *sh;
@@ -101 +101 @@
  *              and vice versa.
  */
-int sheet_insert(sheet_t *sh, spt_t *pos, enum dir_spec dir, char *str)
+errno_t sheet_insert(sheet_t *sh, spt_t *pos, enum dir_spec dir, char *str)
 {
         char *ipp;
@@ -147 +147 @@
  * @return      EOK on success or an error code.
  **/
-int sheet_delete(sheet_t *sh, spt_t *spos, spt_t *epos)
+errno_t sheet_delete(sheet_t *sh, spt_t *spos, spt_t *epos)
 {
         char *spp;

uspace/dist/src/c/demos/edit/sheet.h

@@ -90 +90 @@
 } tag_t;
 
-extern int sheet_create(sheet_t **);
-extern int sheet_insert(sheet_t *, spt_t *, enum dir_spec, char *);
-extern int sheet_delete(sheet_t *, spt_t *, spt_t *);
+extern errno_t sheet_create(sheet_t **);
+extern errno_t sheet_insert(sheet_t *, spt_t *, enum dir_spec, char *);
+extern errno_t sheet_delete(sheet_t *, spt_t *, spt_t *);
 extern void sheet_copy_out(sheet_t *, spt_t const *, spt_t const *, char *,
     size_t, spt_t *);