Changeset eb13ef8 in mainline for uspace/lib/c/generic/io/con_srv.c

Timestamp:

2019-02-06T13:25:12Z (6 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:

fafb8e5

Parents:

bb97118

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-02 14:04:02)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-06 13:25:12)

Message:

Change IPC_GET_* and IPC_SET_* to accept pointer instead of lvalue

File:

• uspace/lib/c/generic/io/con_srv.c (modified) (8 diffs)

uspace/lib/c/generic/io/con_srv.c

@@ -44 +44 @@
 static errno_t console_ev_encode(cons_event_t *event, ipc_call_t *icall)
 {
-        IPC_SET_ARG1(*icall, event->type);
+        IPC_SET_ARG1(icall, event->type);
 
         switch (event->type) {
         case CEV_KEY:
-                IPC_SET_ARG2(*icall, event->ev.key.type);
-                IPC_SET_ARG3(*icall, event->ev.key.key);
-                IPC_SET_ARG4(*icall, event->ev.key.mods);
-                IPC_SET_ARG5(*icall, event->ev.key.c);
+                IPC_SET_ARG2(icall, event->ev.key.type);
+                IPC_SET_ARG3(icall, event->ev.key.key);
+                IPC_SET_ARG4(icall, event->ev.key.mods);
+                IPC_SET_ARG5(icall, event->ev.key.c);
                 break;
         case CEV_POS:
-                IPC_SET_ARG2(*icall, (event->ev.pos.pos_id << 16) | (event->ev.pos.type & 0xffff));
-                IPC_SET_ARG3(*icall, event->ev.pos.btn_num);
-                IPC_SET_ARG4(*icall, event->ev.pos.hpos);
-                IPC_SET_ARG5(*icall, event->ev.pos.vpos);
+                IPC_SET_ARG2(icall, (event->ev.pos.pos_id << 16) | (event->ev.pos.type & 0xffff));
+                IPC_SET_ARG3(icall, event->ev.pos.btn_num);
+                IPC_SET_ARG4(icall, event->ev.pos.hpos);
+                IPC_SET_ARG5(icall, event->ev.pos.vpos);
                 break;
         default:
@@ -158 +158 @@
         sysarg_t row;
 
-        col = IPC_GET_ARG1(*icall);
-        row = IPC_GET_ARG2(*icall);
+        col = IPC_GET_ARG1(icall);
+        row = IPC_GET_ARG2(icall);
 
         if (srv->srvs->ops->set_pos == NULL) {
@@ -218 +218 @@
         console_style_t style;
 
-        style = IPC_GET_ARG1(*icall);
+        style = IPC_GET_ARG1(icall);
 
         if (srv->srvs->ops->set_style == NULL) {
@@ -235 +235 @@
         console_color_attr_t flags;
 
-        bgcolor = IPC_GET_ARG1(*icall);
-        fgcolor = IPC_GET_ARG2(*icall);
-        flags = IPC_GET_ARG3(*icall);
+        bgcolor = IPC_GET_ARG1(icall);
+        fgcolor = IPC_GET_ARG2(icall);
+        flags = IPC_GET_ARG3(icall);
 
         if (srv->srvs->ops->set_color == NULL) {
@@ -253 +253 @@
         pixel_t fgcolor;
 
-        bgcolor = IPC_GET_ARG1(*icall);
-        fgcolor = IPC_GET_ARG2(*icall);
+        bgcolor = IPC_GET_ARG1(icall);
+        fgcolor = IPC_GET_ARG2(icall);
 
         if (srv->srvs->ops->set_rgb_color == NULL) {
@@ -269 +269 @@
         bool show;
 
-        show = IPC_GET_ARG1(*icall);
+        show = IPC_GET_ARG1(icall);
 
         if (srv->srvs->ops->set_cursor_visibility == NULL) {
@@ -303 +303 @@
         }
 
-        async_answer_5(icall, rc, IPC_GET_ARG1(result), IPC_GET_ARG2(result),
-            IPC_GET_ARG3(result), IPC_GET_ARG4(result), IPC_GET_ARG5(result));
+        async_answer_5(icall, rc, IPC_GET_ARG1(&result), IPC_GET_ARG2(&result),
+            IPC_GET_ARG3(&result), IPC_GET_ARG4(&result), IPC_GET_ARG5(&result));
 }
 
@@ -364 +364 @@
                         break;
 
-                sysarg_t method = IPC_GET_IMETHOD(call);
+                sysarg_t method = IPC_GET_IMETHOD(&call);
 
                 if (!method) {