sim: constify dup_arg_p
authorMike Frysinger <vapier@gentoo.org>
Mon, 12 Apr 2010 16:00:44 +0000 (16:00 +0000)
committerMike Frysinger <vapier@gentoo.org>
Mon, 12 Apr 2010 16:00:44 +0000 (16:00 +0000)
The dup_arg_p function is only given const strings, and it maintains a
local table of those const strings.  So constify the whole thing to fix
the GCC warnings:

common/sim-options.c: In function 'sim_parse_args':
common/sim-options.c:559: warning: passing argument 1 of 'dup_arg_p'
discards qualifiers from pointer target type
common/sim-options.c: In function 'print_help':
common/sim-options.c:675: warning: passing argument 1 of 'dup_arg_p'
discards qualifiers from pointer target type

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
sim/common/ChangeLog
sim/common/sim-options.c

index 1fef069935b57287ba834d50922b0a2281aa468a..1bf564af84cdd3b30a28d44499b12e13740ec20e 100644 (file)
@@ -1,3 +1,8 @@
+2010-04-12  Mike Frysinger  <vapier@gentoo.org>
+
+       * sim-options.c (dup_arg_p): Add "const" to the "arg" argument,
+       the local "arg_table" variable, and the xmalloc cast.
+
 2010-04-10  Mike Frysinger  <vapier@gentoo.org>
 
        * sim-fpu.c (sim_fpu_print_status): Add const markings to local
index a37dca9a4cc3d29c6b134b65c2a743b740b744c8..02ad40e60b58d1e201c149e7c3ae1f18be0e3db4 100644 (file)
@@ -489,15 +489,15 @@ standard_install (SIM_DESC sd)
 #define ARG_HASH(a) ((256 * (unsigned char) a[0] + (unsigned char) a[1]) % ARG_HASH_SIZE)
 
 static int
-dup_arg_p (char *arg)
+dup_arg_p (const char *arg)
 {
   int hash;
-  static char **arg_table = NULL;
+  static const char **arg_table = NULL;
 
   if (arg == NULL)
     {
       if (arg_table == NULL)
-       arg_table = (char **) xmalloc (ARG_HASH_SIZE * sizeof (char *));
+       arg_table = (const char **) xmalloc (ARG_HASH_SIZE * sizeof (char *));
       memset (arg_table, 0, ARG_HASH_SIZE * sizeof (char *));
       return 0;
     }