]> cvs.zerfleddert.de Git - hmcfgusb/commitdiff
Cleanup: Improve const correctness
authorFlorian Franzmann <bwlf@bandrate.org>
Sat, 5 Sep 2020 14:33:08 +0000 (16:33 +0200)
committerMichael Gernoth <michael@gernoth.net>
Sat, 5 Sep 2020 16:41:13 +0000 (18:41 +0200)
culfw.c
culfw.h
hexdump.h
hmcfgusb.c
hmland.c
hmsniff.c

diff --git a/culfw.c b/culfw.c
index 34d3eba90c29188a06e4b0469a47266c4b8f20ca..99b20cb248f97758726d8d9faf3c2ee1589a8264 100644 (file)
--- a/culfw.c
+++ b/culfw.c
@@ -110,7 +110,7 @@ out:
        return NULL;
 }
 
-int culfw_send(struct culfw_dev *dev, char *cmd, int cmdlen)
+int culfw_send(struct culfw_dev *dev, const char *cmd, int cmdlen)
 {
        int w = 0;
        int ret;
diff --git a/culfw.h b/culfw.h
index bbe64f54c376a1e58537f1d30350075b4e809a88..9c98eb7d5250f0d5ae711efd98b627674fba8290 100644 (file)
--- a/culfw.h
+++ b/culfw.h
@@ -32,7 +32,7 @@ struct culfw_dev {
 };
 
 struct culfw_dev *culfw_init(char *device, uint32_t speed, culfw_cb_fn cb, void *data);
-int culfw_send(struct culfw_dev *dev, char *cmd, int cmdlen);
+int culfw_send(struct culfw_dev *dev, const char *cmd, int cmdlen);
 int culfw_poll(struct culfw_dev *dev, int timeout);
 void culfw_close(struct culfw_dev *dev);
 void culfw_flush(struct culfw_dev *dev);
index d3d4f3b5dc16d4d6ce9c1f5fe0cd3b2fa84e9dc3..127bfb090aeb6c1caec4955b2a9d9b6d105dcf23 100644 (file)
--- a/hexdump.h
+++ b/hexdump.h
@@ -21,7 +21,7 @@
  * IN THE SOFTWARE.
  */
 
-static void asciishow(unsigned char *buf, int len)
+static void asciishow(const unsigned char *buf, int len)
 {
        int i;
 
@@ -35,7 +35,7 @@ static void asciishow(unsigned char *buf, int len)
        }
 }
 
-static void hexdump(unsigned char *buf, int len, char *prefix)
+static void hexdump(const unsigned char *buf, int len, const char *prefix)
 {
        int i, j;
 
index b9b0f2347df0dd9b7b283e7004cd85ed014fbb4a..94733828d604bdd01e28d6bf353f47e3e4d12437 100644 (file)
@@ -61,7 +61,7 @@ static int debug = 0;
 static int libusb_initialized = 0;
 
 /* Not in all libusb-1.0 versions, so we have to roll our own :-( */
-static char * usb_strerror(int e)
+static const char* usb_strerror(int e)
 {
        static char unknerr[256];
 
@@ -99,7 +99,7 @@ static char * usb_strerror(int e)
        return unknerr;
 }
 
-static char * usb_str_transfer_status(int e)
+static const char* usb_str_transfer_status(int e)
 {
        static char unknerr[256];
 
index 38e4d6baa499a03536a5bfd95ac0c25bbc1dcbe1..380c99a02c2a16fa2c328fd5807d3757cd8665f8 100644 (file)
--- a/hmland.c
+++ b/hmland.c
@@ -100,7 +100,7 @@ static void print_timestamp(FILE *f)
        fprintf(f, "%s.%06ld: ", ts, tv.tv_usec);
 }
 
-static void write_log(char *buf, int len, char *fmt, ...)
+static void write_log(const char *buf, int len, const char *fmt, ...)
 {
        va_list ap;
        int i;
index 01afe05788f00b41597bb0c02d691de687708174..2ffb9e112fb445c2285991197463deb8833571fb 100644 (file)
--- a/hmsniff.c
+++ b/hmsniff.c
@@ -42,7 +42,7 @@
 static int verbose = 0;
 
 /* See HMConfig.pm */
-char *hm_message_types(uint8_t type, uint8_t subtype)
+const char *hm_message_types(uint8_t type, uint8_t subtype)
 {
        switch(type) {
                case 0x00:
Impressum, Datenschutz