ldlinux: Sprinkle const type qualifers
authorMatt Fleming <matt.fleming@linux.intel.com>
Wed, 16 Mar 2011 13:26:57 +0000 (13:26 +0000)
committerMatt Fleming <matt.fleming@linux.intel.com>
Wed, 16 Mar 2011 14:06:39 +0000 (14:06 +0000)
The following warnings are all caused because we're assigning from
const char * to char *. As the strings are never modified, it makes
sense to add the const type qualifer instead of casting away const.

readconfig.c: In function ‘parse_config_file’:
readconfig.c:708:11: warning: assignment discards qualifiers from pointer target type
readconfig.c:937:10: warning: assignment discards qualifiers from pointer target type
readconfig.c:987:20: warning: assignment discards qualifiers from pointer target type
readconfig.c:998:15: warning: assignment discards qualifiers from pointer target type
readconfig.c:1004:18: warning: assignment discards qualifiers from pointer target type
readconfig.c:1041:11: warning: assignment discards qualifiers from pointer target type

Signed-off-by: Matt Fleming <matt.fleming@linux.intel.com>
com32/elflink/ldlinux/config.h
com32/elflink/ldlinux/readconfig.c
core/elflink/core-elf.h
core/elflink/kernel.c

index 7765266..37c57da 100644 (file)
@@ -33,7 +33,7 @@ extern short vkernel;         //have we seen any "label" statements?
 extern short displaycon;       //conio.inc
 extern short nohalt;           //idle.inc
 
-extern char *default_cmd;      //"default" command line
-extern char *onerror;          //"onerror" command line
+extern const char *default_cmd;        //"default" command line
+extern const char *onerror;    //"onerror" command line
 
 #endif /* __CONFIG_H__ */
index c618dc0..b6cfb1b 100644 (file)
@@ -59,8 +59,8 @@ short vkernel = 0;            //have we seen any "label" statements?
 short displaycon = 1;          //conio.inc
 short nohalt = 1;              //idle.inc
 
-char *default_cmd = NULL;      //"default" command line
-char *onerror = NULL;          //"onerror" command line
+const char *default_cmd = NULL;        //"default" command line
+const char *onerror = NULL;    //"onerror" command line
 
 /* Empty refstring */
 const char *empty_string;
@@ -75,7 +75,7 @@ long long totaltimeout = 0;
 
 /* Keep track of global default */
 static int has_ui = 0;         /* DEFAULT only counts if UI is found */
-extern char *globaldefault;
+extern const char *globaldefault;
 static bool menusave = false;  /* True if there is any "menu save" */
 
 /* Linked list of all entires, hidden or not; used by unlabel() */
@@ -568,7 +568,7 @@ uint32_t parse_argb(char **p)
  * same way as if the files had been concatenated together.
  */
 //static const char *append = NULL;
-extern char *append;
+extern const char *append;
 //static unsigned int ipappend = 0;
 unsigned int ipappend = 0;
 static struct labeldata ld;
index d13f506..dcca900 100644 (file)
@@ -18,9 +18,9 @@ enum kernel_type {
     KT_CONFIG,                 /* Configuration file */
 };
 
-extern char *append;
+extern const char *append;
 extern char *ippappend;
-extern char *globaldefault;
+extern const char *globaldefault;
 extern short onerrorlen;
 
 extern int new_linux_kernel(char *okernel, char *ocmdline);
index 6cb3363..3ce2358 100644 (file)
@@ -10,8 +10,8 @@
 #include "core.h"
 #include "core-elf.h"
 
-char *globaldefault = NULL;
-char *append = NULL;
+const char *globaldefault = NULL;
+const char *append = NULL;
 
 /* Will be called from readconfig.c */
 int new_linux_kernel(char *okernel, char *ocmdline)