sim: use libiberty countargv in more places
authorMike Frysinger <vapier@gentoo.org>
Sun, 3 Jan 2016 06:40:46 +0000 (01:40 -0500)
committerMike Frysinger <vapier@gentoo.org>
Sun, 3 Jan 2016 09:08:56 +0000 (04:08 -0500)
A bunch of places open code the countargv implementation, or outright
duplicate it (as count_argc).  Replace all of those w/countargv.

12 files changed:
sim/arm/ChangeLog
sim/arm/wrapper.c
sim/bfin/ChangeLog
sim/bfin/interp.c
sim/common/ChangeLog
sim/common/sim-options.c
sim/erc32/ChangeLog
sim/erc32/interf.c
sim/sh/ChangeLog
sim/sh/interp.c
sim/sh64/ChangeLog
sim/sh64/sh64.c

index a0b00f2..e9346c1 100644 (file)
@@ -1,5 +1,10 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * wrapper.c (sim_target_parse_arg_array): Replace for loop with
+       a call to countargv.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete.
        * configure: Regenerate.
 
index 3938f3c..274b294 100644 (file)
@@ -772,12 +772,7 @@ sim_target_parse_command_line (int argc, char ** argv)
 static void
 sim_target_parse_arg_array (char ** argv)
 {
-  int i;
-
-  for (i = 0; argv[i]; i++)
-    ;
-
-  sim_target_parse_command_line (i, argv);
+  sim_target_parse_command_line (countargv (argv), argv);
 }
 
 static sim_cia
index fe7f6a6..ce03a80 100644 (file)
@@ -1,5 +1,11 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * interp.c (count_argc): Delete.
+       (bfin_syscall): Change count_argc to countargv.
+       (bfin_user_init): Likewise.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete.
        * configure: Regenerate.
 
index 10b331f..84fb085 100644 (file)
@@ -112,20 +112,6 @@ static const char cb_libgloss_stat_map_32[] =
 "space,4:st_blksize,4:st_blocks,4:space,8";
 static const char *stat_map_32, *stat_map_64;
 
-/* Count the number of arguments in an argv.  */
-static int
-count_argc (const char * const *argv)
-{
-  int i;
-
-  if (! argv)
-    return -1;
-
-  for (i = 0; argv[i] != NULL; ++i)
-    continue;
-  return i;
-}
-
 /* Simulate a monitor trap, put the result into r0 and errno into r1
    return offset by which to adjust pc.  */
 
@@ -180,12 +166,12 @@ bfin_syscall (SIM_CPU *cpu)
 #ifdef CB_SYS_argc
     case CB_SYS_argc:
       tbuf += sprintf (tbuf, "argc()");
-      sc.result = count_argc (argv);
+      sc.result = countargv ((char **)argv);
       break;
     case CB_SYS_argnlen:
       {
       tbuf += sprintf (tbuf, "argnlen(%u)", args[0]);
-       if (sc.arg1 < count_argc (argv))
+       if (sc.arg1 < countargv ((char **)argv))
          sc.result = strlen (argv[sc.arg1]);
        else
          sc.result = -1;
@@ -194,7 +180,7 @@ bfin_syscall (SIM_CPU *cpu)
     case CB_SYS_argn:
       {
        tbuf += sprintf (tbuf, "argn(%u)", args[0]);
-       if (sc.arg1 < count_argc (argv))
+       if (sc.arg1 < countargv ((char **)argv))
          {
            const char *argn = argv[sc.arg1];
            int len = strlen (argn);
@@ -1073,7 +1059,7 @@ bfin_user_init (SIM_DESC sd, SIM_CPU *cpu, struct bfd *abfd,
   sim_pc_set (cpu, elf_addrs[0]);
 
   /* Figure out how much storage the argv/env strings need.  */
-  argc = count_argc (argv);
+  argc = countargv ((char **)argv);
   if (argc == -1)
     argc = 0;
   argv_flat = argc; /* NUL bytes  */
@@ -1082,7 +1068,7 @@ bfin_user_init (SIM_DESC sd, SIM_CPU *cpu, struct bfd *abfd,
 
   if (!env)
     env = simple_env;
-  envc = count_argc (env);
+  envc = countargv ((char **)env);
   env_flat = envc; /* NUL bytes  */
   for (i = 0; i < envc; ++i)
     env_flat += strlen (env[i]);
index 2e444d1..e66c289 100644 (file)
@@ -1,5 +1,10 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * sim-options.c (sim_parse_args): Replace for loop with a call
+       to countargv.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * nrun.c (myname): Mark const.
        (main): Mark name const.  Replace myname parsing loop with a
        call to lbasename.
index e252756..88663f7 100644 (file)
@@ -474,8 +474,7 @@ sim_parse_args (SIM_DESC sd, char **argv)
   SIM_RC result = SIM_RC_OK;
 
   /* Count the number of arguments.  */
-  for (argc = 0; argv[argc] != NULL; ++argc)
-    continue;
+  argc = countargv (argv);
 
   /* Count the number of options.  */
   num_opts = 0;
index d47549a..17267f4 100644 (file)
@@ -1,5 +1,9 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * interf.c (sim_open): Replace while loop with a call to countargv.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete.
        * configure: Regenerate.
        * exec.c (fpexec): Rename CURRENT_HOST_BYTE_ORDER to
index bfa4f72..7f3b8df 100644 (file)
@@ -171,8 +171,7 @@ sim_open (kind, callback, abfd, argv)
 
     sim_callback = callback;
 
-    while (argv[argc])
-      argc++;
+    argc = countargv (argv);
     while (stat < argc) {
        if (argv[stat][0] == '-') {
            if (strcmp(argv[stat], "-v") == 0) {
index 7fd3492..b3f8854 100644 (file)
@@ -1,5 +1,10 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * interp.c (count_argc): Delete.
+       (trap): Change count_argc to countargv.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete.
        * configure: Regenerate.
        * interp.c (host_little_endian): Change CURRENT_HOST_BYTE_ORDER
index ee34e0d..f59ad00 100644 (file)
@@ -240,20 +240,6 @@ do { \
 #define FPSCR_SZ  ((GET_FPSCR () & FPSCR_MASK_SZ) != 0)
 #define FPSCR_PR  ((GET_FPSCR () & FPSCR_MASK_PR) != 0)
 
-/* Count the number of arguments in an argv.  */
-static int
-count_argc (char **argv)
-{
-  int i;
-
-  if (! argv)
-    return -1;
-
-  for (i = 0; argv[i] != NULL; ++i)
-    continue;
-  return i;
-}
-
 static void
 set_fpscr1 (int x)
 {
@@ -1056,16 +1042,16 @@ trap (SIM_DESC sd, int i, int *regs, unsigned char *insn_ptr,
              break;
            }
          case SYS_argc:
-           regs[0] = count_argc (prog_argv);
+           regs[0] = countargv (prog_argv);
            break;
          case SYS_argnlen:
-           if (regs[5] < count_argc (prog_argv))
+           if (regs[5] < countargv (prog_argv))
              regs[0] = strlen (prog_argv[regs[5]]);
            else
              regs[0] = -1;
            break;
          case SYS_argn:
-           if (regs[5] < count_argc (prog_argv))
+           if (regs[5] < countargv (prog_argv))
              {
                /* Include the termination byte.  */
                int i = strlen (prog_argv[regs[5]]) + 1;
index d719c5c..82a06b0 100644 (file)
@@ -1,5 +1,10 @@
 2016-01-03  Mike Frysinger  <vapier@gentoo.org>
 
+       * sh64.c (count_argc): Delete.
+       (trap_handler): Change count_argc to countargv.
+
+2016-01-03  Mike Frysinger  <vapier@gentoo.org>
+
        * configure.ac (SIM_AC_OPTION_HOSTENDIAN): Delete.
        * configure: Regenerate.
 
index c35b5c1..e239625 100644 (file)
@@ -529,22 +529,6 @@ sh64_pref (SIM_CPU *cpu, SI addr)
   /* TODO: Unimplemented.  */
 }
 
-/* Count the number of arguments.  */
-static int
-count_argc (cpu)
-     SIM_CPU *cpu;
-{
-  int i = 0;
-
-  if (! STATE_PROG_ARGV (CPU_STATE (cpu)))
-    return -1;
-  
-  while (STATE_PROG_ARGV (CPU_STATE (cpu)) [i] != NULL)
-    ++i;
-
-  return i;
-}
-
 /* Read a null terminated string from memory, return in a buffer */
 static char *
 fetch_str (current_cpu, pc, addr)
@@ -634,11 +618,11 @@ trap_handler (SIM_CPU *current_cpu, int shmedia_abi_p, UQI trapnum, PCADDR pc)
            break;
 
          case SYS_argc:
-           SET_H_GR (ret_reg, count_argc (current_cpu));
+           SET_H_GR (ret_reg, countargv (STATE_PROG_ARGV (CPU_STATE (current_cpu))));
            break;
 
          case SYS_argnlen:
-           if (PARM1 < count_argc (current_cpu))
+           if (PARM1 < countargv (STATE_PROG_ARGV (CPU_STATE (current_cpu))))
              SET_H_GR (ret_reg,
                        strlen (STATE_PROG_ARGV (CPU_STATE (current_cpu)) [PARM1]));
            else
@@ -646,7 +630,7 @@ trap_handler (SIM_CPU *current_cpu, int shmedia_abi_p, UQI trapnum, PCADDR pc)
            break;
 
          case SYS_argn:
-           if (PARM1 < count_argc (current_cpu))
+           if (PARM1 < countargv (STATE_PROG_ARGV (CPU_STATE (current_cpu))))
              {
                /* Include the NULL byte.  */
                i = strlen (STATE_PROG_ARGV (CPU_STATE (current_cpu)) [PARM1]) + 1;