]> granicus.if.org Git - curl/commitdiff
tests: fix -Wcast-qual warnings
authorMarcel Raad <raad@teamviewer.com>
Fri, 5 May 2017 19:29:50 +0000 (21:29 +0200)
committerMarcel Raad <raad@teamviewer.com>
Fri, 5 May 2017 19:29:50 +0000 (21:29 +0200)
Avoid casting string literals to non-const char *.

tests/libtest/first.c
tests/libtest/lib506.c
tests/libtest/lib586.c
tests/libtest/test.h
tests/unit/curlcheck.h
tests/unit/unit1395.c

index 231ae368cbbe766fac1d8bd364c6abbcc3557a10..72140e3141294bd6df75e8a77f47d5d377b6199e 100644 (file)
@@ -116,7 +116,7 @@ static void memory_tracking_init(void)
 #endif
 
 /* returns a hexdump in a static memory area */
-char *hexdump(unsigned char *buffer, size_t len)
+char *hexdump(const unsigned char *buffer, size_t len)
 {
   static char dump[200*3+1];
   char *p = dump;
index b8db83cb6830db1d1818d163899331a4428e9dca..d26252b6f233b94d90a0b827f3c81d4c3e0034eb 100644 (file)
@@ -33,7 +33,7 @@ struct Tdata {
 };
 
 struct userdata {
-  char *text;
+  const char *text;
   int counter;
 };
 
@@ -184,7 +184,7 @@ int test(char *URL)
   int i;
   struct userdata user;
 
-  user.text = (char *)"Pigs in space";
+  user.text = "Pigs in space";
   user.counter = 0;
 
   printf("GLOBAL_INIT\n");
index a4fc6feeeea2f464f92a59e67d36a73eaf13e311..859ef34300365de77446916211f2a6e3bf4f87e3 100644 (file)
@@ -31,7 +31,7 @@ struct Tdata {
 };
 
 struct userdata {
-  char *text;
+  const char *text;
   int counter;
 };
 
@@ -138,7 +138,7 @@ int test(char *URL)
   int i;
   struct userdata user;
 
-  user.text = (char *)"Pigs in space";
+  user.text = "Pigs in space";
   user.counter = 0;
 
   printf("GLOBAL_INIT\n");
index 3da03b8381aa9b6b40bc21be7263a7b9f1b881a5..bb1acca0e295b1b5407a472e41c8e968755086f9 100644 (file)
@@ -67,7 +67,7 @@ extern void wait_ms(int ms); /* wait this many milliseconds */
 extern int test(char *URL); /* the actual test function provided by each
                                individual libXXX.c file */
 
-extern char *hexdump(unsigned char *buffer, size_t len);
+extern char *hexdump(const unsigned char *buffer, size_t len);
 
 #ifdef UNITTESTS
 extern int unitfail;
index 0660e2bedd40ae8a1910b883dfd70c284b1a75cc..46cb84aaff78f27c3ae9f6e4bb85b587e4506f55 100644 (file)
 #define verify_memory(dynamic, check, len)                                  \
   if(dynamic && memcmp(dynamic, check, len)) {                              \
     fprintf(stderr, "%s:%d Memory buffer mismatch size %d. '%s' is not\n",  \
-            __FILE__, __LINE__, len, hexdump((unsigned char *)check, len)); \
-    fprintf(stderr, "%s:%d the same as '%s'\n",                             \
-            __FILE__, __LINE__, hexdump((unsigned char *)dynamic, len));    \
+            __FILE__, __LINE__, len,                                        \
+            hexdump((const unsigned char *)check, len));                    \
+    fprintf(stderr, "%s:%d the same as '%s'\n", __FILE__, __LINE__,         \
+            hexdump((const unsigned char *)dynamic, len));                  \
     unitfail++;                                                             \
   }
 
index 13f464134fed4ff4506018df948b2d868257b24e..ba8e14f45e3a29b34146f215abd9a02e636270e4 100644 (file)
@@ -68,7 +68,7 @@ UNITTEST_START
   };
 
   for(i=0; i < sizeof(pairs)/sizeof(pairs[0]); i++) {
-    char *out = Curl_dedotdotify((char *)pairs[i].input);
+    char *out = Curl_dedotdotify(pairs[i].input);
     abort_unless(out != NULL, "returned NULL!");
 
     if(strcmp(out, pairs[i].output)) {