]> sigrok.org Git - libsigrok.git/commitdiff
Constify a lot more items.
authorUwe Hermann <redacted>
Fri, 6 Nov 2015 17:18:34 +0000 (18:18 +0100)
committerUwe Hermann <redacted>
Sun, 8 Nov 2015 18:11:25 +0000 (19:11 +0100)
This fixes various compiler warnings when -Wdiscarded-qualifiers is used.

29 files changed:
include/libsigrok/libsigrok.h
src/analog.c
src/hardware/atten-pps3xxx/protocol.c
src/hardware/atten-pps3xxx/protocol.h
src/hardware/baylibre-acme/protocol.c
src/hardware/center-3xx/protocol.h
src/hardware/fx2lafw/api.c
src/hardware/hantek-dso/api.c
src/hardware/hantek-dso/dso.h
src/hardware/kern-scale/protocol.h
src/hardware/korad-kdxxxxp/protocol.c
src/hardware/korad-kdxxxxp/protocol.h
src/hardware/lascar-el-usb/protocol.h
src/hardware/manson-hcs-3xxx/protocol.h
src/hardware/mic-985xx/protocol.h
src/hardware/serial-dmm/protocol.h
src/hardware/testo/protocol.c
src/hardware/testo/protocol.h
src/hardware/uni-t-dmm/protocol.h
src/hardware/zeroplus-logic-cube/api.c
src/hwdriver.c
src/input/binary.c
src/input/chronovu_la8.c
src/input/csv.c
src/input/input.c
src/input/vcd.c
src/libsigrok-internal.h
src/output/analog.c
src/scpi/scpi_vxi.c

index 1030f258fed69ccc066ad6d9dd0695f38435b74e..5309efbdbb55fd34c426a6ab5e14f8df030e9e3c 100644 (file)
@@ -546,11 +546,11 @@ struct sr_analog_spec {
 /** Generic option struct used by various subsystems. */
 struct sr_option {
        /* Short name suitable for commandline usage, [a-z0-9-]. */
-       char *id;
+       const char *id;
        /* Short name suitable for GUI usage, can contain UTF-8. */
-       char *name;
+       const char *name;
        /* Description of the option, in a sentence. */
-       char *desc;
+       const char *desc;
        /* Default value for this option. */
        GVariant *def;
        /* List of possible values, if this is an option with few values. */
@@ -645,11 +645,11 @@ struct sr_key_info {
        /** Data type like SR_T_STRING, etc if applicable. */
        int datatype;
        /** Short, lowercase ID string, e.g. "serialcomm", "voltage". */
-       char *id;
+       const char *id;
        /** Full capitalized name, e.g. "Serial communication". */
-       char *name;
+       const char *name;
        /** Verbose description (unused currently). */
-       char *description;
+       const char *description;
 };
 
 #define SR_CONF_GET  (1 << 31)
index 8d87302afbfc9ee090943ca0331437e7ed8306ec..f26f7a5c2b931e0afa855cd0b106fa401ca7c3cf 100644 (file)
@@ -45,7 +45,7 @@
 
 struct unit_mq_string {
        uint64_t value;
-       char *str;
+       const char *str;
 };
 
 /* Please use the same order as in enum sr_unit (libsigrok.h). */
index a969e9763cd1f19a8e205147bcbe664c1ca66148..34e0af42824ebff7fa13623c79e84bfb8ad93f9d 100644 (file)
@@ -21,7 +21,7 @@
 #include <string.h>
 #include "protocol.h"
 
-static void dump_packet(char *msg, uint8_t *packet)
+static void dump_packet(const char *msg, uint8_t *packet)
 {
        int i;
        char str[128];
index 633c0a20d39835f7beb3be79b430ba92bc3b98a9..b9a256d17ab820daf73e631890a70bc6ad05278b 100644 (file)
@@ -54,7 +54,7 @@ struct channel_spec {
 
 struct pps_model {
        int modelid;
-       char *name;
+       const char *name;
        int channel_modes;
        int num_channels;
        struct channel_spec channels[MAX_CHANNELS];
index becd68517cb215f66205b657c30fe2e788ed7852..6726667ed78d3d29888d4d4bb8ed927de8608de6 100644 (file)
@@ -651,7 +651,8 @@ static float read_sample(struct sr_channel *ch)
 SR_PRIV int bl_acme_open_channel(struct sr_channel *ch)
 {
        struct channel_priv *chp;
-       char path[64], *file;
+       char path[64];
+       const char *file;
        int fd;
 
        chp = ch->priv;
index 8ff5d2cc801c35fd521ad40dd512dd90a0af62d3..acad71d4ece59ed4001d5673701ae7f0a7246bf2 100644 (file)
@@ -37,9 +37,9 @@ enum {
 };
 
 struct center_dev_info {
-       char *vendor;
-       char *device;
-       char *conn;
+       const char *vendor;
+       const char *device;
+       const char *conn;
        int num_channels;
        uint32_t max_sample_points;
        uint8_t packet_size;
index 6376d0fb43103cc64a3ead264c90eaf5510028a7..c3f3ed4b5e89406e3f6d207a9850fb9ce4441a81 100644 (file)
@@ -374,7 +374,7 @@ static int dev_open(struct sr_dev_inst *sdi)
        struct sr_dev_driver *di = sdi->driver;
        struct sr_usb_dev_inst *usb;
        struct dev_context *devc;
-       char *fpga_firmware = NULL;
+       const char *fpga_firmware = NULL;
        int ret;
        int64_t timediff_us, timediff_ms;
 
index 7eef397a38b1ff986a22acd2aa76b39acdf3f41c..1c9289e5712efbf40a7aaf5127fb0443a4f65d71 100644 (file)
@@ -428,7 +428,8 @@ static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *s
 {
        struct dev_context *devc;
        struct sr_usb_dev_inst *usb;
-       char str[128], *s;
+       char str[128];
+       const char *s;
        const uint64_t *vdiv;
        int ch_idx;
 
index 7938e9e516d597037c0e61985df16c970e9e1df4..44cced48400605bad020554ade3fd3e950ab764e 100644 (file)
@@ -154,10 +154,10 @@ struct dso_profile {
        /* VID/PID after firmware upload */
        uint16_t fw_vid;
        uint16_t fw_pid;
-       char *vendor;
-       char *model;
+       const char *vendor;
+       const char *model;
        const uint64_t *buffersizes;
-       char *firmware;
+       const char *firmware;
 };
 
 struct dev_context {
index 9be375f7b8867d69b9912c02f8bdf80ed78ad6b6..fce47003a669e68dbd4ce6076ce507532e5eec2a 100644 (file)
@@ -27,11 +27,11 @@ struct scale_info {
        /** libsigrok driver info struct. */
        struct sr_dev_driver di;
        /** Manufacturer/brand. */
-       char *vendor;
+       const char *vendor;
        /** Model. */
-       char *device;
+       const char *device;
        /** serialconn string. */
-       char *conn;
+       const char *conn;
        /** Baud rate. */
        uint32_t baudrate;
        /** Packet size in bytes. */
index 182908a8553ff254ee716347088cab39b0622fd7..bec63b4e40c5ad375a30e262ac5eebf6053b93ec 100644 (file)
@@ -80,7 +80,8 @@ static void give_device_time_to_process(struct dev_context *devc)
 SR_PRIV int korad_kdxxxxp_set_value(struct sr_serial_dev_inst *serial,
                                struct dev_context *devc)
 {
-       char msg[21], *cmd;
+       char msg[21];
+       const char *cmd;
        float value;
        int ret;
 
index 5b7ee1c35782beebb655707aee5b099ca5ed261b..a76b28bae3d50a2475ebc0fdf64789f008cf74bf 100644 (file)
@@ -44,9 +44,9 @@ enum {
 /* Information on single model */
 struct korad_kdxxxxp_model {
        int model_id; /**< Model info */
-       char *vendor; /**< Vendor name */
-       char *name; /**< Model name */
-       char *id; /**< Model ID, as delivered by interface */
+       const char *vendor; /**< Vendor name */
+       const char *name; /**< Model name */
+       const char *id; /**< Model ID, as delivered by interface */
        int channels; /**< Number of channels */
        double voltage[3]; /**< Min, max, step */
        double current[3]; /**< Min, max, step */
index 08197fb202be065a9d0f7303b8460f1b7cbfb40a..482c4a9788fd923445a9ba3178346ddaa355a49f 100644 (file)
@@ -68,7 +68,7 @@ enum {
 
 struct elusb_profile {
        int modelid;
-       char *modelname;
+       const char *modelname;
        int logformat;
 };
 
index c63f943d7645679f285440621df847f386994ddf..8e614ce2a335621932a9869c99358d6aea5ac268 100644 (file)
@@ -58,8 +58,8 @@ enum {
 /** Information on a single model. */
 struct hcs_model {
        int model_id;      /**< Model info */
-       char *name;        /**< Model name */
-       char *id;          /**< Model ID, like delivered by interface */
+       const char *name;  /**< Model name */
+       const char *id;    /**< Model ID, like delivered by interface */
        double voltage[3]; /**< Min, max, step */
        double current[3]; /**< Min, max, step */
 };
index 42a01daf6151166e25f849e7a1b5c97c69eb21da..f0e31a61e8cf5bcded1b4054c45c7422644ede2c 100644 (file)
@@ -36,9 +36,9 @@ enum {
 };
 
 struct mic_dev_info {
-       char *vendor;
-       char *device;
-       char *conn;
+       const char *vendor;
+       const char *device;
+       const char *conn;
        uint32_t max_sample_points;
        gboolean has_temperature;
        gboolean has_humidity;
index 1a0b5939a5fe786ab91a7a8e53e3037144b36303..3c2d88b80876cf111cc25e9f25696dc31ec952b6 100644 (file)
@@ -26,11 +26,11 @@ struct dmm_info {
        /** libsigrok driver info struct. */
        struct sr_dev_driver di;
        /** Manufacturer/brand. */
-       char *vendor;
+       const char *vendor;
        /** Model. */
-       char *device;
+       const char *device;
        /** serialconn string. */
-       char *conn;
+       const char *conn;
        /** Baud rate. */
        uint32_t baudrate;
        /** Packet size in bytes. */
index e2c43c080667784a5b31867bedb61bd26dff2371..be36023f7bd56849b74e3b960cd78ba60a878af1 100644 (file)
@@ -61,7 +61,7 @@ SR_PRIV int testo_probe_channels(struct sr_dev_inst *sdi)
        struct sr_usb_dev_inst *usb;
        int unit, packet_len, len, i;
        unsigned char packet[MAX_REPLY_SIZE], buf[MAX_REPLY_SIZE];
-       char *probe_name;
+       const char *probe_name;
 
        devc = sdi->priv;
        usb = sdi->conn;
index 8621bc2cc1f2512595eb6838ad18700308da7598..06a29644c17b0f646a0c727651304849281e25bd 100644 (file)
@@ -46,7 +46,7 @@
 #define EP_OUT               2 | LIBUSB_ENDPOINT_OUT
 
 struct testo_model {
-       char *name;
+       const char *name;
        int request_size;
        const uint8_t *request;
 };
index 9844f20ec0b22039e7d9cfecdafeb5685adfaee4..f8fdbb5ad6c111b37282f2f0e2e1df5ec33551c0 100644 (file)
@@ -31,8 +31,8 @@
 
 struct dmm_info {
        struct sr_dev_driver di;
-       char *vendor;
-       char *device;
+       const char *vendor;
+       const char *device;
        uint32_t baudrate;
        int packet_size;
        gboolean (*packet_valid)(const uint8_t *);
index 50b7df9e423ece744797a9af375bf7af8403a9f3..6a76b3d8d758cd14806bcd6e17eafe4aafd35b63 100644 (file)
@@ -31,7 +31,7 @@
 struct zp_model {
        uint16_t vid;
        uint16_t pid;
-       char *model_name;
+       const char *model_name;
        unsigned int channels;
        unsigned int sample_depth;      /* In Ksamples/channel */
        unsigned int max_sampling_freq;
index c5d4bf29b02ded8aa387f8266ef129ef400fb9ab..b15fc28fc9d3f0083099f0e201e7a50c51e3ae83 100644 (file)
@@ -569,7 +569,8 @@ static int check_key(const struct sr_dev_driver *driver,
        GVariant *gvar_opts;
        const uint32_t *opts;
        uint32_t pub_opt;
-       char *suffix, *opstr;
+       const char *suffix;
+       const char *opstr;
 
        if (sdi && cg)
                suffix = " for this device and channel group";
index 736e4cf24e043133161be90c9bf4735716d06f98..8c5a19fd4cc116cc6ce913b7297d7e9b7ee3e997 100644 (file)
@@ -150,7 +150,7 @@ static struct sr_option options[] = {
        ALL_ZERO
 };
 
-static struct sr_option *get_options(void)
+static const struct sr_option *get_options(void)
 {
        if (!options[0].def) {
                options[0].def = g_variant_ref_sink(g_variant_new_int32(DEFAULT_NUM_CHANNELS));
index 53e1477321510583c7529d60c8411ccc4c5b4b01..1491e0425884706a0961f12a6c5b4db372f61038 100644 (file)
@@ -162,7 +162,7 @@ static struct sr_option options[] = {
        ALL_ZERO
 };
 
-static struct sr_option *get_options(void)
+static const struct sr_option *get_options(void)
 {
        if (!options[0].def) {
                options[0].def = g_variant_ref_sink(g_variant_new_int32(DEFAULT_NUM_CHANNELS));
index cc451bacdab05382b5a38f9074e56f77c8d4ef00..7b884674730e044e925cabb7bc0263c15df2cb40 100644 (file)
@@ -784,7 +784,7 @@ static struct sr_option options[] = {
        ALL_ZERO
 };
 
-static struct sr_option *get_options(void)
+static const struct sr_option *get_options(void)
 {
        if (!options[0].def) {
                options[0].def = g_variant_ref_sink(g_variant_new_int32(0));
index 5e8f2129676f1f2839953a7c5dd797debbd1fca9..9d471ca4fdad53612889036dec3d0c498d00c6df 100644 (file)
@@ -242,7 +242,7 @@ SR_API struct sr_input *sr_input_new(const struct sr_input_module *imod,
                GHashTable *options)
 {
        struct sr_input *in;
-       struct sr_option *mod_opts;
+       const struct sr_option *mod_opts;
        const GVariantType *gvt;
        GHashTable *new_opts;
        GHashTableIter iter;
index 75efc0d55efe023c6ae1d1f482c3766a3bcc4ecb..6089ac8434a1369d3a8302ee81998f3accef980f 100644 (file)
@@ -551,7 +551,7 @@ static struct sr_option options[] = {
        ALL_ZERO
 };
 
-static struct sr_option *get_options(void)
+static const struct sr_option *get_options(void)
 {
        if (!options[0].def) {
                options[0].def = g_variant_ref_sink(g_variant_new_int32(DEFAULT_NUM_CHANNELS));
index 4f44b53b70c6a871f9aa6f435c9a6bf8ba152b1f..84ad295ef61de0ee1204e83c54299668cf6fe1b6 100644 (file)
@@ -288,7 +288,7 @@ struct sr_input_module {
         * Returns a NULL-terminated list of options this module can take.
         * Can be NULL, if the module has no options.
         */
-       struct sr_option *(*options) (void);
+       const struct sr_option *(*options) (void);
 
        /**
         * Check if this input module can load and parse the specified stream.
@@ -383,7 +383,7 @@ struct sr_output_module {
         * A unique ID for this output module, suitable for use in command-line
         * clients, [a-z0-9-]. Must not be NULL.
         */
-       char *id;
+       const char *id;
 
        /**
         * A unique name for this output module, suitable for use in GUI
@@ -397,7 +397,7 @@ struct sr_output_module {
         * This can be displayed by frontends, e.g. when selecting the output
         * module for saving a file.
         */
-       char *desc;
+       const char *desc;
 
        /**
         * A NULL terminated array of strings containing a list of file name
@@ -491,7 +491,7 @@ struct sr_transform_module {
         * A unique ID for this transform module, suitable for use in
         * command-line clients, [a-z0-9-]. Must not be NULL.
         */
-       char *id;
+       const char *id;
 
        /**
         * A unique name for this transform module, suitable for use in GUI
@@ -505,7 +505,7 @@ struct sr_transform_module {
         * This can be displayed by frontends, e.g. when selecting
         * which transform module(s) to add.
         */
-       char *desc;
+       const char *desc;
 
        /**
         * Returns a NULL-terminated list of options this transform module
index 88b85d18eddf81b489c011a94918ecb9efb30da5..ed480ca0335385af204804cad0f5d0003b228da6 100644 (file)
@@ -68,7 +68,7 @@ static int init(struct sr_output *o, GHashTable *options)
        return SR_OK;
 }
 
-static void si_printf(float value, GString *out, char *unitstr)
+static void si_printf(float value, GString *out, const char *unitstr)
 {
        float v;
 
index 26cd832dba4ab85d9309ca00bb7ceaea047090b2..8a99ec7474b3e5cee740f9e6367e0faa4651d315 100644 (file)
@@ -76,7 +76,7 @@ static int scpi_vxi_open(struct sr_scpi_dev_inst *scpi)
        link_parms.clientId = (long) vxi->client;
        link_parms.lockDevice = 0;
        link_parms.lock_timeout = VXI_DEFAULT_TIMEOUT_MS;
-       link_parms.device = "inst0";
+       link_parms.device = (char *)"inst0";
 
        if (!(link_resp = create_link_1(&link_parms, vxi->client))) {
                sr_err("Link creation failed for %s", vxi->address);