Changeset 46577995 in mainline for uspace/lib/gui/terminal.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/lib/gui/terminal.c (modified) (9 diffs)

uspace/lib/gui/terminal.c

@@ -62 +62 @@
 static LIST_INITIALIZE(terms);
 
-static int term_open(con_srvs_t *, con_srv_t *);
-static int term_close(con_srv_t *);
-static int term_read(con_srv_t *, void *, size_t, size_t *);
-static int term_write(con_srv_t *, void *, size_t, size_t *);
+static errno_t term_open(con_srvs_t *, con_srv_t *);
+static errno_t term_close(con_srv_t *);
+static errno_t term_read(con_srv_t *, void *, size_t, size_t *);
+static errno_t term_write(con_srv_t *, void *, size_t, size_t *);
 static void term_sync(con_srv_t *);
 static void term_clear(con_srv_t *);
 static void term_set_pos(con_srv_t *, sysarg_t col, sysarg_t row);
-static int term_get_pos(con_srv_t *, sysarg_t *, sysarg_t *);
-static int term_get_size(con_srv_t *, sysarg_t *, sysarg_t *);
-static int term_get_color_cap(con_srv_t *, console_caps_t *);
+static errno_t term_get_pos(con_srv_t *, sysarg_t *, sysarg_t *);
+static errno_t term_get_size(con_srv_t *, sysarg_t *, sysarg_t *);
+static errno_t term_get_color_cap(con_srv_t *, console_caps_t *);
 static void term_set_style(con_srv_t *, console_style_t);
 static void term_set_color(con_srv_t *, console_color_t, console_color_t,
@@ -77 +77 @@
 static void term_set_rgb_color(con_srv_t *, pixel_t, pixel_t);
 static void term_set_cursor_visibility(con_srv_t *, bool);
-static int term_get_event(con_srv_t *, cons_event_t *);
+static errno_t term_get_event(con_srv_t *, cons_event_t *);
 
 static con_ops_t con_ops = {
@@ -376 +376 @@
 }
 
-static int term_open(con_srvs_t *srvs, con_srv_t *srv)
+static errno_t term_open(con_srvs_t *srvs, con_srv_t *srv)
 {
         return EOK;
 }
 
-static int term_close(con_srv_t *srv)
+static errno_t term_close(con_srv_t *srv)
 {
         return EOK;
 }
 
-static int term_read(con_srv_t *srv, void *buf, size_t size, size_t *nread)
+static errno_t term_read(con_srv_t *srv, void *buf, size_t size, size_t *nread)
 {
         terminal_t *term = srv_to_terminal(srv);
@@ -463 +463 @@
 }
 
-static int term_write(con_srv_t *srv, void *data, size_t size, size_t *nwritten)
+static errno_t term_write(con_srv_t *srv, void *data, size_t size, size_t *nwritten)
 {
         terminal_t *term = srv_to_terminal(srv);
@@ -504 +504 @@
 }
 
-static int term_get_pos(con_srv_t *srv, sysarg_t *col, sysarg_t *row)
+static errno_t term_get_pos(con_srv_t *srv, sysarg_t *col, sysarg_t *row)
 {
         terminal_t *term = srv_to_terminal(srv);
@@ -515 +515 @@
 }
 
-static int term_get_size(con_srv_t *srv, sysarg_t *cols, sysarg_t *rows)
+static errno_t term_get_size(con_srv_t *srv, sysarg_t *cols, sysarg_t *rows)
 {
         terminal_t *term = srv_to_terminal(srv);
@@ -527 +527 @@
 }
 
-static int term_get_color_cap(con_srv_t *srv, console_caps_t *caps)
+static errno_t term_get_color_cap(con_srv_t *srv, console_caps_t *caps)
 {
         (void) srv;
@@ -575 +575 @@
 }
 
-static int term_get_event(con_srv_t *srv, cons_event_t *event)
+static errno_t term_get_event(con_srv_t *srv, cons_event_t *event)
 {
         terminal_t *term = srv_to_terminal(srv);
@@ -751 +751 @@
         term->srvs.sarg = term;
         
-        int rc = loc_server_register(NAME);
+        errno_t rc = loc_server_register(NAME);
         if (rc != EOK) {
                 widget_deinit(&term->widget);