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

uspace/lib/gui/window.c

@@ -163 +163 @@
         
         font_t *font;
-        int rc = embedded_font_create(&font, 16);
+        errno_t rc = embedded_font_create(&font, 16);
         if (rc != EOK) {
                 window_yield(widget->window);
@@ -404 +404 @@
         
         /* Inform compositor about new surface. */
-        int rc = win_resize(win->osess, offset_x, offset_y, width, height,
+        errno_t rc = win_resize(win->osess, offset_x, offset_y, width, height,
             placement_flags, surface_direct_access(new_surface));
         
@@ -493 +493 @@
 
 /* Window event loop. Runs in own dedicated fibril. */
-static int event_loop(void *arg)
+static errno_t event_loop(void *arg)
 {
         bool is_main = false;
@@ -561 +561 @@
 
 /* Input fetcher from compositor. Runs in own dedicated fibril. */
-static int fetch_input(void *arg)
-{
-        int rc;
+static errno_t fetch_input(void *arg)
+{
+        errno_t rc;
         bool terminate = false;
         window_t *win = (window_t *) arg;
@@ -618 +618 @@
         
         service_id_t reg_dsid;
-        int rc = loc_service_get_id(winreg, &reg_dsid, 0);
+        errno_t rc = loc_service_get_id(winreg, &reg_dsid, 0);
         if (rc != EOK) {
                 free(win);
@@ -677 +677 @@
 }
 
-int window_set_caption(window_t *win, const char *caption)
+errno_t window_set_caption(window_t *win, const char *caption)
 {
         char *cap;