Changeset 84239b1 in mainline for uspace/lib


Ignore:
Timestamp:
2018-03-11T19:39:11Z (8 years ago)
Author:
Jiri Svoboda <jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
f3d47c97
Parents:
850fd32
Message:

And there was much fixing.

Location:
uspace/lib
Files:
12 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/bithenge/src/compound.c

    r850fd32 r84239b1  
    7070
    7171        /* i ranges from (self->num - 1) to 0 inside the loop. */
    72         for (size_t i = self->num; i--; ) {
     72        size_t i = self->num;
     73        while (i-- != 0) {
    7374                bithenge_node_t *tmp;
    7475                rc = bithenge_transform_apply(self->xforms[i], scope, in,

  • uspace/lib/bithenge/src/print.c

    r850fd32 r84239b1  
    162162        const char *value = bithenge_string_node_value(node);
    163163        state_printf(state, "\"");
    164         for (string_iterator_t i = string_iterator(value); !string_iterator_done(&i); ) {
     164        string_iterator_t i = string_iterator(value);
     165        while (!string_iterator_done(&i)) {
    165166                wchar_t ch;
    166167                errno_t rc = string_iterator_next(&i, &ch);

  • uspace/lib/bithenge/src/sequence.c

    r850fd32 r84239b1  
    607607        self->subtransforms = subtransforms;
    608608        self->num_subtransforms = 0;
    609         for (self->num_subtransforms = 0;
    610             subtransforms[self->num_subtransforms].transform;
    611             self->num_subtransforms++);
     609        while (subtransforms[self->num_subtransforms].transform)
     610            self->num_subtransforms++;
    612611        *out = struct_as_transform(self);
    613612        return EOK;

  • uspace/lib/c/generic/device/hw_res_parsed.c

    r850fd32 r84239b1  
    167167                        size_t s_size;
    168168
    169                         s_address = RNGABS(out->mem_ranges.ranges[i]);;
     169                        s_address = RNGABS(out->mem_ranges.ranges[i]);
    170170                        s_size = RNGSZ(out->mem_ranges.ranges[i]);
    171171
    … …  
    237237
    238238        return EOK;
    239 };
     239}
    240240
    241241/** Get hw_resources from the parent device.
    … …  
    279279
    280280        return rc;
    281 };
     281}
    282282
    283283/** @}

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

    r850fd32 r84239b1  
    15171517                                counter += retval;
    15181518                                j = nxt;
    1519                                 goto next_char;
     1519                                continue;
    15201520                        case 'c':
    15211521                                if (qualifier == PrintfQualifierLong)
    … …  
    15271527                                        counter = -counter;
    15281528                                        goto out;
    1529                                 };
     1529                                }
    15301530
    15311531                                counter += retval;
    15321532                                j = nxt;
    1533                                 goto next_char;
     1533                                continue;
    15341534
    15351535                        /*
    … …  
    15521552                                counter += retval;
    15531553                                j = nxt;
    1554                                 goto next_char;
     1554                                continue;
    15551555
    15561556                        /*
    … …  
    15891589                        case '%':
    15901590                                j = i;
    1591                                 goto next_char;
     1591                                continue;
    15921592
    15931593                        /*
    … …  
    15991599                                 * so we will print whole bad format sequence.
    16001600                                 */
    1601                                 goto next_char;
     1601                                continue;
    16021602                        }
    16031603
    … …  
    16551655                        j = nxt;
    16561656                }
    1657 next_char:
    1658                 ;
    16591657        }
    16601658

  • uspace/lib/ext4/src/balloc.c

    r850fd32 r84239b1  
    352352        uint32_t free_blocks;
    353353        uint32_t goal;
     354        uint32_t block_size;
    354355
    355356        /* Find GOAL */
    … …  
    585586
    586587success:
    587         /* Empty command - because of syntax */
    588         ;
    589 
    590         uint32_t block_size = ext4_superblock_get_block_size(sb);
     588        block_size = ext4_superblock_get_block_size(sb);
    591589
    592590        /* Update superblock free blocks count */

  • uspace/lib/ext4/src/extent.c

    r850fd32 r84239b1  
    476476        uint16_t pos = 0;
    477477        errno_t rc;
     478        errno_t rc2;
    478479        while (ext4_extent_header_get_depth(eh) != 0) {
    479480                /* Search index in index node by iblock */
    … …  
    513514
    514515cleanup:
    515         ;
    516 
    517         errno_t rc2 = EOK;
     516        rc2 = EOK;
    518517
    519518        /*
    … …  
    623622        /* Find the first extent to modify */
    624623        ext4_extent_path_t *path;
     624        errno_t rc2;
    625625        errno_t rc = ext4_extent_find_extent(inode_ref, iblock_from, &path);
    626626        if (rc != EOK)
    … …  
    734734
    735735cleanup:
    736         ;
    737 
    738         errno_t rc2 = EOK;
     736        rc2 = EOK;
    739737
    740738        /*
    … …  
    982980        /* Load the nearest leaf (with extent) */
    983981        ext4_extent_path_t *path;
     982        errno_t rc2;
    984983        errno_t rc = ext4_extent_find_extent(inode_ref, new_block_idx, &path);
    985984        if (rc != EOK)
    … …  
    10861085
    10871086finish:
    1088         ;
    1089 
    1090         errno_t rc2 = EOK;
     1087        rc2 = EOK;
    10911088
    10921089        /* Set return values */

  • uspace/lib/ext4/src/ops.c

    r850fd32 r84239b1  
    219219        ext4_node_t *eparent = EXT4_NODE(pfn);
    220220        ext4_filesystem_t *fs = eparent->instance->filesystem;
     221        errno_t rc2;
    221222
    222223        if (!ext4_inode_is_type(fs->superblock, eparent->inode_ref->inode,
    … …  
    244245
    245246exit:
    246         ;
    247 
    248247        /* Destroy search result structure */
    249         errno_t const rc2 = ext4_directory_destroy_result(&result);
     248        rc2 = ext4_directory_destroy_result(&result);
    250249        return rc == EOK ? rc2 : rc;
    251250}
    … …  
    12791278{
    12801279        fs_node_t *fn;
     1280        errno_t rc2;
    12811281        errno_t rc = ext4_node_get(&fn, service_id, index);
    12821282        if (rc != EOK)
    … …  
    13951395
    13961396exit:
    1397         ;
    1398 
    1399         errno_t const rc2 = ext4_node_put(fn);
     1397        rc2 = ext4_node_put(fn);
    14001398        return rc == EOK ? rc2 : rc;
    14011399}

  • uspace/lib/hound/src/protocol.c

    r850fd32 r84239b1  
    7373                uint8_t channels;
    7474                uint8_t format;
    75         } f __attribute__((packed));
     75        } __attribute__((packed)) f;
    7676        sysarg_t arg;
    7777} format_convert_t;
    … …  
    385385void hound_connection_handler(ipc_callid_t iid, ipc_call_t *icall, void *arg)
    386386{
     387        hound_context_id_t id;
     388        errno_t ret;
     389        int flags;
     390        void *source;
     391        void *sink;
     392
    387393        /* Accept connection if there is a valid iface*/
    388394        if (server_iface) {
    … …  
    397403                ipc_callid_t callid = async_get_call(&call);
    398404                switch (IPC_GET_IMETHOD(call)) {
    399                 case IPC_M_HOUND_CONTEXT_REGISTER: {
     405                case IPC_M_HOUND_CONTEXT_REGISTER:
    400406                        /* check interface functions */
    401407                        if (!server_iface || !server_iface->add_context) {
    … …  
    407413
    408414                        /* Get context name */
    409                         errno_t ret =
    410                             async_data_write_accept(&name, true, 0, 0, 0, 0);
     415                        ret = async_data_write_accept(&name, true, 0, 0, 0, 0);
    411416                        if (ret != EOK) {
    412417                                async_answer_0(callid, ret);
    413418                                break;
    414419                        }
    415                         hound_context_id_t id = 0;
     420
     421                        id = 0;
    416422                        ret = server_iface->add_context(server_iface->server,
    417423                            &id, name, record);
    … …  
    424430                        }
    425431                        break;
    426                 }
    427                 case IPC_M_HOUND_CONTEXT_UNREGISTER: {
     432                case IPC_M_HOUND_CONTEXT_UNREGISTER:
    428433                        /* check interface functions */
    429434                        if (!server_iface || !server_iface->rem_context) {
    … …  
    433438
    434439                        /* get id, 1st param */
    435                         hound_context_id_t id = IPC_GET_ARG1(call);
    436                         const errno_t ret =
    437                             server_iface->rem_context(server_iface->server, id);
     440                        id = IPC_GET_ARG1(call);
     441                        ret = server_iface->rem_context(server_iface->server,
     442                            id);
    438443                        async_answer_0(callid, ret);
    439444                        break;
    440                 }
    441                 case IPC_M_HOUND_GET_LIST: {
     445                case IPC_M_HOUND_GET_LIST:
    442446                        /* check interface functions */
    443447                        if (!server_iface || !server_iface->get_list) {
    … …  
    447451
    448452                        char **list = NULL;
    449                         const int flags = IPC_GET_ARG1(call);
     453                        flags = IPC_GET_ARG1(call);
    450454                        size_t count = IPC_GET_ARG2(call);
    451455                        const bool conn = IPC_GET_ARG3(call);
    452456                        char *conn_name = NULL;
    453                         errno_t ret = EOK;
     457                        ret = EOK;
    454458
    455459                        /* get connected actor name if provided */
    … …  
    501505                        free(list);
    502506                        break;
    503                 }
    504                 case IPC_M_HOUND_CONNECT: {
     507                case IPC_M_HOUND_CONNECT:
    505508                        /* check interface functions */
    506509                        if (!server_iface || !server_iface->connect) {
    … …  
    509512                        }
    510513
    511                         void *source = NULL;
    512                         void *sink = NULL;
     514                        source = NULL;
     515                        sink = NULL;
    513516
    514517                        /* read source name */
    515                         errno_t ret =
    516                             async_data_write_accept(&source, true, 0, 0, 0, 0);
     518                        ret = async_data_write_accept(&source, true, 0, 0, 0,
     519                            0);
    517520                        /* read sink name */
    518521                        if (ret == EOK)
    … …  
    527530                        async_answer_0(callid, ret);
    528531                        break;
    529                 }
    530                 case IPC_M_HOUND_DISCONNECT: {
     532                case IPC_M_HOUND_DISCONNECT:
    531533                        /* check interface functions */
    532534                        if (!server_iface || !server_iface->disconnect) {
    … …  
    535537                        }
    536538
    537                         void *source = NULL;
    538                         void *sink = NULL;
     539                        source = NULL;
     540                        sink = NULL;
    539541
    540542                        /* read source name */
    541                         errno_t ret =
    542                             async_data_write_accept(&source, true, 0, 0, 0, 0);
     543                        ret = async_data_write_accept(&source, true, 0, 0, 0,
     544                            0);
    543545                        /*read sink name */
    544546                        if (ret == EOK)
    … …  
    552554                        async_answer_0(callid, ret);
    553555                        break;
    554                 }
    555                 case IPC_M_HOUND_STREAM_ENTER: {
     556                case IPC_M_HOUND_STREAM_ENTER:
    556557                        /* check interface functions */
    557558                        if (!server_iface || !server_iface->is_record_context
    … …  
    563564
    564565                        /* get parameters */
    565                         hound_context_id_t id = IPC_GET_ARG1(call);
    566                         const int flags = IPC_GET_ARG2(call);
     566                        id = IPC_GET_ARG1(call);
     567                        flags = IPC_GET_ARG2(call);
    567568                        const format_convert_t c = {.arg = IPC_GET_ARG3(call)};
    568569                        const pcm_format_t f = {
    … …  
    574575
    575576                        void *stream;
    576                         errno_t ret = server_iface->add_stream(server_iface->server,
     577                        ret = server_iface->add_stream(server_iface->server,
    577578                            id, flags, f, bsize, &stream);
    578579                        if (ret != EOK) {
    … …  
    604605                        }
    605606                        break;
    606                 }
    607607                case IPC_M_HOUND_STREAM_EXIT:
    608608                case IPC_M_HOUND_STREAM_DRAIN:

  • uspace/lib/math/include/mathtypes.h

    r850fd32 r84239b1  
    9696                uint32_t exp : 8;
    9797                uint32_t fraction : 23;
    98         } parts __attribute__((packed));
     98        } __attribute__((packed)) parts;
    9999} float32;
    100100
    … …  
    106106                uint64_t exp : 11;
    107107                uint64_t fraction : 52;
    108         } parts __attribute__((packed));
     108        } __attribute__((packed)) parts;
    109109} float64;
    110110
    … …  
    113113                uint64_t hi;
    114114                uint32_t lo;
    115         } bin __attribute__((packed));
     115        } __attribute__((packed)) bin;
    116116
    117117        struct {
    … …  
    120120                uint64_t exp : 15;
    121121                uint64_t fraction : 64;
    122         } parts __attribute__((packed));
     122        } __attribute__((packed)) parts;
    123123} float96;
    124124
    … …  
    127127                uint64_t hi;
    128128                uint64_t lo;
    129         } bin __attribute__((packed));
     129        } __attribute__((packed)) bin;
    130130
    131131        struct {
    … …  
    134134                uint64_t frac_hi : 48;
    135135                uint64_t frac_lo : 64;
    136         } parts __attribute__((packed));
     136        } __attribute__((packed)) parts;
    137137} float128;
    138138
    … …  
    146146                uint32_t exp : 8;
    147147                uint32_t sign : 1;
    148         } parts __attribute__((packed));
     148        } __attribute__((packed)) parts;
    149149} float32;
    150150
    … …  
    156156                uint64_t exp : 11;
    157157                uint64_t sign : 1;
    158         } parts __attribute__((packed));
     158        } __attribute__((packed)) parts;
    159159} float64;
    160160
    … …  
    163163                uint32_t lo;
    164164                uint64_t hi;
    165         } bin __attribute__((packed));
     165        } __attribute__((packed)) bin;
    166166
    167167        struct {
    … …  
    170170                uint64_t sign : 1;
    171171                uint64_t padding : 16;
    172         } parts __attribute__((packed));
     172        } __attribute__((packed)) parts;
    173173} float96;
    174174
    … …  
    177177                uint64_t lo;
    178178                uint64_t hi;
    179         } bin __attribute__((packed));
     179        } __attribute__((packed)) bin;
    180180
    181181        struct {
    … …  
    184184                uint64_t exp : 15;
    185185                uint64_t sign : 1;
    186         } parts __attribute__((packed));
     186        } __attribute__((packed)) parts;
    187187} float128;
    188188

  • uspace/lib/nic/src/nic_driver.c

    r850fd32 r84239b1  
    389389                *period = nic_data->poll_period;
    390390        return nic_data->poll_mode;
    391 };
     391}
    392392
    393393/** Inform the NICF about poll mode
    … …  
    487487
    488488        memcpy(addr, &nic_data->mac, sizeof(nic_address_t));
    489 };
     489}
    490490
    491491/**

  • uspace/lib/pcm/src/format.c

    r850fd32 r84239b1  
    304304        case PCM_SAMPLE_SINT24_BE:
    305305        case PCM_SAMPLE_FLOAT32:
    306         default: ;
     306        default:
     307                break;
    307308        }
    308309        return 0;
Note: See TracChangeset for help on using the changeset viewer.