Changeset 05b59393 in mainline


Ignore:
Timestamp:
2017-10-04T18:02:14Z (7 years ago)
Author:
jzr <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
c9e88da
Parents:
0b2d369
Message:

Fix a couple of benign clang warnings.
No change in semantics.

Location:
uspace
Files:
14 edited

Legend:

Unmodified
Added
Removed

  • uspace/app/bdsh/cmds/builtin_cmds.c

    r0b2d369 r05b59393  
    7979                return (char *)NULL;
    8080
    81         for(i=0; builtin_aliases[i] != NULL; i++) {
     81        for(i=0; builtin_aliases[i] != NULL; i+=2) {
    8282                if (!str_cmp(builtin_aliases[i], command))
    83                         return (char *)builtin_aliases[++i];
    84                 i++;
     83                        return (char *)builtin_aliases[i+1];
    8584        }
    8685

  • uspace/app/bdsh/cmds/mod_cmds.c

    r0b2d369 r05b59393  
    9696                return (char *)NULL;
    9797
    98         for(i=0; mod_aliases[i] != NULL; i++) {
     98        for(i=0; mod_aliases[i] != NULL; i+=2) {
    9999                if (!str_cmp(mod_aliases[i], command))
    100                         return (char *)mod_aliases[++i];
    101                 i++;
     100                        return (char *)mod_aliases[i+1];
    102101        }
    103102

  • uspace/app/fontviewer/fontviewer.c

    r0b2d369 r05b59393  
    7878                                points += increment;
    7979               
    80                         if ((event->key == KC_DOWN)) {
     80                        if (event->key == KC_DOWN) {
    8181                                if (points <= increment) {
    8282                                        points = 1;

  • uspace/app/sbi/src/builtin/bi_task.c

    r0b2d369 r05b59393  
    114114        }
    115115
    116         cmd[dim] = '\0';
     116        cmd[dim] = NULL;
    117117
    118118        if (os_exec((char * const *)cmd) != EOK) {

  • uspace/app/sbi/src/run_expr.c

    r0b2d369 r05b59393  
    28862886                                /* Is it static/nonstatic? */
    28872887                                var_sn = stree_symbol_has_attr(
    2888                                     var_to_symbol(var), sac_static);
     2888                                    var_to_symbol(var), sac_static) ? sn_static : sn_nonstatic;
    28892889                                if (var_sn == sn) {
    28902890                                        /* Compute field type. XXX Memoize. */

  • uspace/drv/audio/sb16/dsp.c

    r0b2d369 r05b59393  
    7575                [DSP_NO_BUFFER] = "NO BUFFER",
    7676        };
    77         if (state < ARRAY_SIZE(state_names))
     77        if ((size_t)state < ARRAY_SIZE(state_names))
    7878                return state_names[state];
    7979        return "UNKNOWN";
    … …  
    144144{
    145145        dsp_write(dsp, SET_SAMPLING_RATE_OUTPUT);
    146         dsp_write(dsp, rate >> 8);
    147         dsp_write(dsp, rate & 0xff);
    148         ddf_log_verbose("Sampling rate: %hhx:%hhx.", rate >> 8, rate & 0xff);
     146        uint8_t rate_lo = rate & 0xff;
     147        uint8_t rate_hi = rate >> 8;
     148        dsp_write(dsp, rate_hi);
     149        dsp_write(dsp, rate_lo);
     150        ddf_log_verbose("Sampling rate: %hhx:%hhx.", rate_hi, rate_lo);
    149151}
    150152
    … …  
    202204       
    203205        return ret;
    204 }
    205 
    206 static inline size_t sample_count(pcm_sample_format_t format, size_t byte_count)
    207 {
    208         return byte_count / pcm_sample_format_size(format);
    209206}
    210207

  • uspace/drv/bus/isa/i8237.c

    r0b2d369 r05b59393  
    371371       
    372372        /* 16 bit transfers are a bit special */
    373         ddf_msg(LVL_DEBUG, "Unspoiled address %#" PRIx32 " (size %" PRIu16 ")",
     373        ddf_msg(LVL_DEBUG, "Unspoiled address %#" PRIx32 " (size %" PRIu32 ")",
    374374            pa, size);
    375375        if (is_dma16(channel)) {
    … …  
    388388       
    389389        ddf_msg(LVL_DEBUG, "Setting channel %u to address %#" PRIx32 " "
    390             "(size %" PRIu16 "), mode %hhx.", channel, pa, size, mode);
     390            "(size %" PRIu32 "), mode %hhx.", channel, pa, size, mode);
    391391       
    392392        /* Mask DMA request */

  • uspace/drv/bus/usb/ohci/ohci_rh.c

    r0b2d369 r05b59393  
    367367        case USB_HUB_FEATURE_C_PORT_RESET:        /*20*/
    368368                usb_log_debug2("Clearing port C_CONNECTION, C_ENABLE, "
    369                     "C_SUSPEND, C_OC or C_RESET on port %"PRIu16".\n", port);
     369                    "C_SUSPEND, C_OC or C_RESET on port %u.\n", port);
    370370                /* Bit offsets correspond to the feature number */
    371371                OHCI_WR(hub->registers->rh_port_status[port],
    … …  
    416416        case USB_HUB_FEATURE_PORT_RESET:   /*4*/
    417417                usb_log_debug2("Setting port POWER, ENABLE, SUSPEND or RESET "
    418                     "on port %"PRIu16".\n", port);
     418                    "on port %u.\n", port);
    419419                /* Bit offsets correspond to the feature number */
    420420                OHCI_WR(hub->registers->rh_port_status[port], 1 << feature);

  • uspace/drv/fb/amdm37x_dispc/amdm37x_dispc.c

    r0b2d369 r05b59393  
    267267        amdm37x_dispc_t *dispc = vis->dev_ctx;
    268268        const visual_t visual = mode.cell_visual.pixel_visual;
    269         assert(visual < sizeof(pixel2visual_table) / sizeof(pixel2visual_table[0]));
     269        assert((size_t)visual < sizeof(pixel2visual_table) / sizeof(pixel2visual_table[0]));
    270270        const unsigned bpp = pixel2visual_table[visual].bpp;
    271271        pixel2visual_t p2v = pixel2visual_table[visual].func;

  • uspace/lib/c/generic/cap.c

    r0b2d369 r05b59393  
    185185        sunit = NULL;
    186186
    187         if (cap->cunit < 0 || cap->cunit >= CU_LIMIT)
    188                 assert(false);
     187        assert(cap->cunit < CU_LIMIT);
    189188
    190189        rc = ipow10_u64(cap->dp, &div);

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

    r0b2d369 r05b59393  
    4848
    4949/** Stream structure */
    50 typedef struct hound_stream {
     50struct hound_stream {
    5151        /** link in context's list */
    5252        link_t link;
    … …  
    5959        /** Stream flags */
    6060        int flags;
    61 } hound_stream_t;
     61};
    6262
    6363/**
    … …  
    7272
    7373/** Hound client context structure */
    74 typedef struct hound_context {
     74struct hound_context {
    7575        /** Audio session */
    7676        hound_sess_t *session;
    … …  
    8989        /** Assigned context id */
    9090        hound_context_id_t id;
    91 } hound_context_t;
     91};
    9292
    9393/**

  • uspace/lib/nettl/src/amap.c

    r0b2d369 r05b59393  
    5858#include <stdint.h>
    5959#include <stdlib.h>
     60
     61static portrng_flags_t aflags_to_pflags(amap_flags_t flags)
     62{
     63        // FIXME: either use a single type, or provide a proper conversion
     64        return (portrng_flags_t) flags;
     65}
    6066
    6167/** Create association map.
    … …  
    378384        mepp = *epp;
    379385
    380         rc = portrng_alloc(repla->portrng, epp->local.port, arg, flags,
     386        rc = portrng_alloc(repla->portrng, epp->local.port, arg, aflags_to_pflags(flags),
    381387            &mepp.local.port);
    382388        if (rc != EOK) {
    … …  
    422428        mepp = *epp;
    423429
    424         rc = portrng_alloc(laddr->portrng, epp->local.port, arg, flags,
     430        rc = portrng_alloc(laddr->portrng, epp->local.port, arg, aflags_to_pflags(flags),
    425431            &mepp.local.port);
    426432        if (rc != EOK) {
    … …  
    466472        mepp = *epp;
    467473
    468         rc = portrng_alloc(llink->portrng, epp->local.port, arg, flags,
     474        rc = portrng_alloc(llink->portrng, epp->local.port, arg, aflags_to_pflags(flags),
    469475            &mepp.local.port);
    470476        if (rc != EOK) {
    … …  
    498504        mepp = *epp;
    499505
    500         rc = portrng_alloc(map->unspec, epp->local.port, arg, flags,
     506        rc = portrng_alloc(map->unspec, epp->local.port, arg, aflags_to_pflags(flags),
    501507            &mepp.local.port);
    502508        if (rc != EOK) {

  • uspace/lib/usb/src/usb.c

    r0b2d369 r05b59393  
    7373const char *usb_str_transfer_type(usb_transfer_type_t t)
    7474{
    75         if (t >= ARRAY_SIZE(str_transfer_type)) {
     75        if ((size_t)t >= ARRAY_SIZE(str_transfer_type)) {
    7676                return "invalid";
    7777        }
    … …  
    8686const char *usb_str_transfer_type_short(usb_transfer_type_t t)
    8787{
    88         if (t >= ARRAY_SIZE(str_transfer_type_short)) {
     88        if ((size_t)t >= ARRAY_SIZE(str_transfer_type_short)) {
    8989                return "invl";
    9090        }

  • uspace/lib/usbdev/src/devdrv.c

    r0b2d369 r05b59393  
    5353
    5454/** USB device structure. */
    55 typedef struct usb_device {
     55struct usb_device {
    5656        /** Connection to device on USB bus */
    5757        usb_dev_session_t *bus_session;
    … …  
    9595         */
    9696        void *driver_data;
    97 } usb_device_t;
     97};
    9898
    9999/** Count number of pipes the driver expects.
Note: See TracChangeset for help on using the changeset viewer.