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


Ignore:
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

Location:
uspace/lib/c/generic/io
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/c/generic/io/chardev.c

    rbb97118 reb13ef8  
    125125        }
    126126
    127         *nread = IPC_GET_ARG2(answer);
     127        *nread = IPC_GET_ARG2(&answer);
    128128        /* In case of partial success, ARG1 contains the error code */
    129         return (errno_t) IPC_GET_ARG1(answer);
     129        return (errno_t) IPC_GET_ARG1(&answer);
    130130
    131131}
    … …  
    176176        }
    177177
    178         *nwritten = IPC_GET_ARG2(answer);
     178        *nwritten = IPC_GET_ARG2(&answer);
    179179        /* In case of partial success, ARG1 contains the error code */
    180         return (errno_t) IPC_GET_ARG1(answer);
     180        return (errno_t) IPC_GET_ARG1(&answer);
    181181}
    182182

  • uspace/lib/c/generic/io/chardev_srv.c

    rbb97118 reb13ef8  
    5151        errno_t rc;
    5252
    53         flags = IPC_GET_ARG1(*icall);
     53        flags = IPC_GET_ARG1(icall);
    5454
    5555        ipc_call_t call;
    … …  
    154154                ipc_call_t call;
    155155                async_get_call(&call);
    156                 sysarg_t method = IPC_GET_IMETHOD(call);
     156                sysarg_t method = IPC_GET_IMETHOD(&call);
    157157
    158158                if (!method) {

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

    rbb97118 reb13ef8  
    4444static errno_t console_ev_encode(cons_event_t *event, ipc_call_t *icall)
    4545{
    46         IPC_SET_ARG1(*icall, event->type);
     46        IPC_SET_ARG1(icall, event->type);
    4747
    4848        switch (event->type) {
    4949        case CEV_KEY:
    50                 IPC_SET_ARG2(*icall, event->ev.key.type);
    51                 IPC_SET_ARG3(*icall, event->ev.key.key);
    52                 IPC_SET_ARG4(*icall, event->ev.key.mods);
    53                 IPC_SET_ARG5(*icall, event->ev.key.c);
     50                IPC_SET_ARG2(icall, event->ev.key.type);
     51                IPC_SET_ARG3(icall, event->ev.key.key);
     52                IPC_SET_ARG4(icall, event->ev.key.mods);
     53                IPC_SET_ARG5(icall, event->ev.key.c);
    5454                break;
    5555        case CEV_POS:
    56                 IPC_SET_ARG2(*icall, (event->ev.pos.pos_id << 16) | (event->ev.pos.type & 0xffff));
    57                 IPC_SET_ARG3(*icall, event->ev.pos.btn_num);
    58                 IPC_SET_ARG4(*icall, event->ev.pos.hpos);
    59                 IPC_SET_ARG5(*icall, event->ev.pos.vpos);
     56                IPC_SET_ARG2(icall, (event->ev.pos.pos_id << 16) | (event->ev.pos.type & 0xffff));
     57                IPC_SET_ARG3(icall, event->ev.pos.btn_num);
     58                IPC_SET_ARG4(icall, event->ev.pos.hpos);
     59                IPC_SET_ARG5(icall, event->ev.pos.vpos);
    6060                break;
    6161        default:
    … …  
    158158        sysarg_t row;
    159159
    160         col = IPC_GET_ARG1(*icall);
    161         row = IPC_GET_ARG2(*icall);
     160        col = IPC_GET_ARG1(icall);
     161        row = IPC_GET_ARG2(icall);
    162162
    163163        if (srv->srvs->ops->set_pos == NULL) {
    … …  
    218218        console_style_t style;
    219219
    220         style = IPC_GET_ARG1(*icall);
     220        style = IPC_GET_ARG1(icall);
    221221
    222222        if (srv->srvs->ops->set_style == NULL) {
    … …  
    235235        console_color_attr_t flags;
    236236
    237         bgcolor = IPC_GET_ARG1(*icall);
    238         fgcolor = IPC_GET_ARG2(*icall);
    239         flags = IPC_GET_ARG3(*icall);
     237        bgcolor = IPC_GET_ARG1(icall);
     238        fgcolor = IPC_GET_ARG2(icall);
     239        flags = IPC_GET_ARG3(icall);
    240240
    241241        if (srv->srvs->ops->set_color == NULL) {
    … …  
    253253        pixel_t fgcolor;
    254254
    255         bgcolor = IPC_GET_ARG1(*icall);
    256         fgcolor = IPC_GET_ARG2(*icall);
     255        bgcolor = IPC_GET_ARG1(icall);
     256        fgcolor = IPC_GET_ARG2(icall);
    257257
    258258        if (srv->srvs->ops->set_rgb_color == NULL) {
    … …  
    269269        bool show;
    270270
    271         show = IPC_GET_ARG1(*icall);
     271        show = IPC_GET_ARG1(icall);
    272272
    273273        if (srv->srvs->ops->set_cursor_visibility == NULL) {
    … …  
    303303        }
    304304
    305         async_answer_5(icall, rc, IPC_GET_ARG1(result), IPC_GET_ARG2(result),
    306             IPC_GET_ARG3(result), IPC_GET_ARG4(result), IPC_GET_ARG5(result));
     305        async_answer_5(icall, rc, IPC_GET_ARG1(&result), IPC_GET_ARG2(&result),
     306            IPC_GET_ARG3(&result), IPC_GET_ARG4(&result), IPC_GET_ARG5(&result));
    307307}
    308308
    … …  
    364364                        break;
    365365
    366                 sysarg_t method = IPC_GET_IMETHOD(call);
     366                sysarg_t method = IPC_GET_IMETHOD(&call);
    367367
    368368                if (!method) {

  • uspace/lib/c/generic/io/console.c

    rbb97118 reb13ef8  
    156156static errno_t console_ev_decode(ipc_call_t *call, cons_event_t *event)
    157157{
    158         event->type = IPC_GET_ARG1(*call);
     158        event->type = IPC_GET_ARG1(call);
    159159
    160160        switch (event->type) {
    161161        case CEV_KEY:
    162                 event->ev.key.type = IPC_GET_ARG2(*call);
    163                 event->ev.key.key = IPC_GET_ARG3(*call);
    164                 event->ev.key.mods = IPC_GET_ARG4(*call);
    165                 event->ev.key.c = IPC_GET_ARG5(*call);
     162                event->ev.key.type = IPC_GET_ARG2(call);
     163                event->ev.key.key = IPC_GET_ARG3(call);
     164                event->ev.key.mods = IPC_GET_ARG4(call);
     165                event->ev.key.c = IPC_GET_ARG5(call);
    166166                break;
    167167        case CEV_POS:
    168                 event->ev.pos.pos_id = IPC_GET_ARG2(*call) >> 16;
    169                 event->ev.pos.type = IPC_GET_ARG2(*call) & 0xffff;
    170                 event->ev.pos.btn_num = IPC_GET_ARG3(*call);
    171                 event->ev.pos.hpos = IPC_GET_ARG4(*call);
    172                 event->ev.pos.vpos = IPC_GET_ARG5(*call);
     168                event->ev.pos.pos_id = IPC_GET_ARG2(call) >> 16;
     169                event->ev.pos.type = IPC_GET_ARG2(call) & 0xffff;
     170                event->ev.pos.btn_num = IPC_GET_ARG3(call);
     171                event->ev.pos.hpos = IPC_GET_ARG4(call);
     172                event->ev.pos.vpos = IPC_GET_ARG5(call);
    173173                break;
    174174        default:

  • uspace/lib/c/generic/io/input.c

    rbb97118 reb13ef8  
    112112        errno_t rc;
    113113
    114         type = IPC_GET_ARG1(*call);
    115         key = IPC_GET_ARG2(*call);
    116         mods = IPC_GET_ARG3(*call);
    117         c = IPC_GET_ARG4(*call);
     114        type = IPC_GET_ARG1(call);
     115        key = IPC_GET_ARG2(call);
     116        mods = IPC_GET_ARG3(call);
     117        c = IPC_GET_ARG4(call);
    118118
    119119        rc = input->ev_ops->key(input, type, key, mods, c);
    … …  
    127127        errno_t rc;
    128128
    129         dx = IPC_GET_ARG1(*call);
    130         dy = IPC_GET_ARG2(*call);
     129        dx = IPC_GET_ARG1(call);
     130        dy = IPC_GET_ARG2(call);
    131131
    132132        rc = input->ev_ops->move(input, dx, dy);
    … …  
    142142        errno_t rc;
    143143
    144         x = IPC_GET_ARG1(*call);
    145         y = IPC_GET_ARG2(*call);
    146         max_x = IPC_GET_ARG3(*call);
    147         max_y = IPC_GET_ARG4(*call);
     144        x = IPC_GET_ARG1(call);
     145        y = IPC_GET_ARG2(call);
     146        max_x = IPC_GET_ARG3(call);
     147        max_y = IPC_GET_ARG4(call);
    148148
    149149        rc = input->ev_ops->abs_move(input, x, y, max_x, max_y);
    … …  
    157157        errno_t rc;
    158158
    159         bnum = IPC_GET_ARG1(*call);
    160         press = IPC_GET_ARG2(*call);
     159        bnum = IPC_GET_ARG1(call);
     160        press = IPC_GET_ARG2(call);
    161161
    162162        rc = input->ev_ops->button(input, bnum, press);
    … …  
    172172                async_get_call(&call);
    173173
    174                 if (!IPC_GET_IMETHOD(call)) {
     174                if (!IPC_GET_IMETHOD(&call)) {
    175175                        async_answer_0(&call, EOK);
    176176                        return;
    177177                }
    178178
    179                 switch (IPC_GET_IMETHOD(call)) {
     179                switch (IPC_GET_IMETHOD(&call)) {
    180180                case INPUT_EVENT_ACTIVE:
    181181                        input_ev_active(input, &call);

  • uspace/lib/c/generic/io/log.c

    rbb97118 reb13ef8  
    208208                return parent;
    209209
    210         return IPC_GET_ARG1(answer);
     210        return IPC_GET_ARG1(&answer);
    211211}
    212212

  • uspace/lib/c/generic/io/output.c

    rbb97118 reb13ef8  
    100100                return 0;
    101101
    102         return (frontbuf_handle_t) IPC_GET_ARG1(answer);
     102        return (frontbuf_handle_t) IPC_GET_ARG1(&answer);
    103103}
    104104
Note: See TracChangeset for help on using the changeset viewer.