sim: use libiberty countargv in more places

A bunch of places open code the countargv implementation, or outright
duplicate it (as count_argc).  Replace all of those w/countargv.

diff --git a/sim/arm/ChangeLog b/sim/arm/ChangeLog
index a0b00f2a800741e1c3e241e5ea54b70178a2f74a..e9346c17321f39f715b880f4dd6cec5b1d89ad95 100644 (file)
--- a/sim/arm/ChangeLog
+++ b/sim/arm/ChangeLog
@@ -1,3 +1,8 @@
+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.

diff --git a/sim/arm/wrapper.c b/sim/arm/wrapper.c
index 3938f3ce5fd8435477fd10b38d22e3199666196c..274b294ecf780252c7441796fb32a72b6c6354c5 100644 (file)
--- a/sim/arm/wrapper.c
+++ b/sim/arm/wrapper.c
@@ -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

diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog
index fe7f6a6e322acbc80e01d563351ac5ad49c055e9..ce03a805badf615cee589b64669fb1567b4913b0 100644 (file)
--- a/sim/bfin/ChangeLog
+++ b/sim/bfin/ChangeLog
@@ -1,3 +1,9 @@
+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.

diff --git a/sim/bfin/interp.c b/sim/bfin/interp.c
index 10b331f55ae16886a0e054cc30abea2e662381d2..84fb0856c44c94e6254639657bf14f331d91bf89 100644 (file)
--- a/sim/bfin/interp.c
+++ b/sim/bfin/interp.c
@@ -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]);

diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index 2e444d133c2b24a302cd10b3f1fafd8ec97a7a72..e66c289cac45f0a9e90c85b958b27527572e9822 100644 (file)
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,8 @@
+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.

diff --git a/sim/common/sim-options.c b/sim/common/sim-options.c
index e25275695fa5fc6618c6a8a61249572bf57b3e43..88663f7030e67bd43b3fe7284e91f6f9e0685e6d 100644 (file)
--- a/sim/common/sim-options.c
+++ b/sim/common/sim-options.c
@@ -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;

diff --git a/sim/erc32/ChangeLog b/sim/erc32/ChangeLog
index d47549a75e280f33ef8660a5ab0bfb721b34844d..17267f49a4b2f2d4287632ec5561a000e5fd1b53 100644 (file)
--- a/sim/erc32/ChangeLog
+++ b/sim/erc32/ChangeLog
@@ -1,3 +1,7 @@
+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.

diff --git a/sim/erc32/interf.c b/sim/erc32/interf.c
index bfa4f72d7ebb6bdeca2d8289ed3f0528ac74f25f..7f3b8dfee8c23ee8bbe52c62d2c26aab2693d693 100644 (file)
--- a/sim/erc32/interf.c
+++ b/sim/erc32/interf.c
@@ -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) {

diff --git a/sim/sh/ChangeLog b/sim/sh/ChangeLog
index 7fd3492411ff0c7c79769cc3fa25e0eb942b6b9a..b3f885481aed2bff4b070349e1fbbba638cfa9f3 100644 (file)
--- a/sim/sh/ChangeLog
+++ b/sim/sh/ChangeLog
@@ -1,3 +1,8 @@
+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.

diff --git a/sim/sh/interp.c b/sim/sh/interp.c
index ee34e0d59281fcf98c0b5e9a7e5c34048de64a4a..f59ad00b5d54b493119a1bea08d1c5496be3c336 100644 (file)
--- a/sim/sh/interp.c
+++ b/sim/sh/interp.c
@@ -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;

diff --git a/sim/sh64/ChangeLog b/sim/sh64/ChangeLog
index d719c5cff68d86dd33f7607bd4ba36bec6e90586..82a06b0b027ae7d50019dcf144ae1f7892bf0642 100644 (file)
--- a/sim/sh64/ChangeLog
+++ b/sim/sh64/ChangeLog
@@ -1,3 +1,8 @@
+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.

diff --git a/sim/sh64/sh64.c b/sim/sh64/sh64.c
index c35b5c1d2a1872a777a0de2619b2a0c4b96bcde4..e239625dc080b33fa4c84d604f1b79ef6cc40566 100644 (file)
--- a/sim/sh64/sh64.c
+++ b/sim/sh64/sh64.c
@@ -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;