[syslinux:elflink] ldlinux: Sprinkle const type qualifers

syslinux-bot for Matt Fleming matt.fleming at linux.intel.com
Fri Mar 18 11:30:49 PDT 2011


Commit-ID:  eee9ce324a85a4974188b93c6ca6a11020d00c1c
Gitweb:     http://syslinux.zytor.com/commit/eee9ce324a85a4974188b93c6ca6a11020d00c1c
Author:     Matt Fleming <matt.fleming at linux.intel.com>
AuthorDate: Wed, 16 Mar 2011 13:26:57 +0000
Committer:  Matt Fleming <matt.fleming at linux.intel.com>
CommitDate: Wed, 16 Mar 2011 14:06:39 +0000

ldlinux: Sprinkle const type qualifers

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 at linux.intel.com>


---
 com32/elflink/ldlinux/config.h     |    4 ++--
 com32/elflink/ldlinux/readconfig.c |    8 ++++----
 core/elflink/core-elf.h            |    4 ++--
 core/elflink/kernel.c              |    4 ++--
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/com32/elflink/ldlinux/config.h b/com32/elflink/ldlinux/config.h
index 7765266..37c57da 100644
--- a/com32/elflink/ldlinux/config.h
+++ b/com32/elflink/ldlinux/config.h
@@ -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__ */
diff --git a/com32/elflink/ldlinux/readconfig.c b/com32/elflink/ldlinux/readconfig.c
index c618dc0..b6cfb1b 100644
--- a/com32/elflink/ldlinux/readconfig.c
+++ b/com32/elflink/ldlinux/readconfig.c
@@ -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;
diff --git a/core/elflink/core-elf.h b/core/elflink/core-elf.h
index d13f506..dcca900 100644
--- a/core/elflink/core-elf.h
+++ b/core/elflink/core-elf.h
@@ -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);
diff --git a/core/elflink/kernel.c b/core/elflink/kernel.c
index 6cb3363..3ce2358 100644
--- a/core/elflink/kernel.c
+++ b/core/elflink/kernel.c
@@ -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)



More information about the Syslinux-commits mailing list