Avoid hacking around argc / argv.

diff --git a/cli/cli_diff.cpp b/cli/cli_diff.cpp
index 0bd1b31f4f2c1c968b86a04d9a9470255023e57d..9f6efefa00eac2ac29e2b086161dd16b0c2a354c 100644 (file)
--- a/cli/cli_diff.cpp
+++ b/cli/cli_diff.cpp
@@ -50,7 +50,7 @@ command(int argc, char *argv[])
 {
     int i;
 
-    for (i = 0; i < argc; ++i) {
+    for (i = 1; i < argc; ++i) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/cli/cli_diff_images.cpp b/cli/cli_diff_images.cpp
index 42aaaff071d6927d73c3c0c36bb3e55e3f5fc222..626a34fc7790f4a56fb5070d81f2246c9deb72db 100644 (file)
--- a/cli/cli_diff_images.cpp
+++ b/cli/cli_diff_images.cpp
@@ -68,12 +68,12 @@ command(int argc, char *argv[])
         return 1;
     }
 
-    char **args = new char* [argc + 2];
+    char **args = new char* [argc + 1];
     args[0] = (char *) command.str();
-    for (i = 0; i < argc; i++) {
-        args[i + 1] = argv[i];
+    for (i = 1; i < argc; i++) {
+        args[i] = argv[i];
     }
-    args[i + 1] = NULL;
+    args[argc] = NULL;
 
     ret = os::execute(args);
 

diff --git a/cli/cli_diff_state.cpp b/cli/cli_diff_state.cpp
index 8918d582f8195023a4e452ad94486bb009a4c82b..8536e3235c16b170388ce899ce9c651d29ae5f7c 100644 (file)
--- a/cli/cli_diff_state.cpp
+++ b/cli/cli_diff_state.cpp
@@ -50,7 +50,7 @@ command(int argc, char *argv[])
 {
     int i;
 
-    for (i = 0; i < argc; ++i) {
+    for (i = 1; i < argc; ++i) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/cli/cli_dump.cpp b/cli/cli_dump.cpp
index 3f393ce18aefcf46aa9a720c993635c1633c6226..d6f50141a3ebda2495dc0eacfddd0415bcdfac55 100644 (file)
--- a/cli/cli_dump.cpp
+++ b/cli/cli_dump.cpp
@@ -117,10 +117,6 @@ command(int argc, char *argv[])
     trace::DumpFlags dumpFlags = 0;
     bool dumpThreadIds = false;
     
-    // FIXME: avoid hacking around argc / argv
-    argc += 1;
-    argv = &argv[-1];
-
     int opt;
     while ((opt = getopt_long(argc, argv, shortOptions, longOptions, NULL)) != -1) {
         switch (opt) {
@@ -154,7 +150,7 @@ command(int argc, char *argv[])
             }
             break;
         case THREAD_IDS_OPT:
-            dumpThreadIds = boolOption(optarg, true);
+            dumpThreadIds = boolOption(optarg);
             break;
         default:
             std::cerr << "error: unexpected option `" << opt << "`\n";

diff --git a/cli/cli_main.cpp b/cli/cli_main.cpp
index a776107fd12a13c6ae2a01cff510245b096af115..cff6a05de111955323219d8575ba7400d955f174 100644 (file)
--- a/cli/cli_main.cpp
+++ b/cli/cli_main.cpp
@@ -122,12 +122,12 @@ do_help_command(int argc, char *argv[])
     const Command *command;
     int i;
 
-    if (argc != 1) {
+    if (argc != 2) {
         help_usage();
         return 0;
     }
 
-    char *command_name = argv[0];
+    char *command_name = argv[1];
 
     for (i = 0; i < ARRAY_SIZE(commands); i++) {
         command = commands[i];
@@ -172,7 +172,7 @@ main(int argc, char **argv)
         return 1;
     }
 
-    command_name = argv[i++];
+    command_name = argv[i];
 
     argc -= i;
     argv = &argv[i];

diff --git a/cli/cli_pickle.cpp b/cli/cli_pickle.cpp
index 8535b18d1ff5aba3a92108f8a737ffba595d3c26..6af879fa3da6960d7aca1a02b2c06cf8e27e2b4a 100644 (file)
--- a/cli/cli_pickle.cpp
+++ b/cli/cli_pickle.cpp
@@ -166,7 +166,7 @@ command(int argc, char *argv[])
 {
     int i;
 
-    for (i = 0; i < argc;) {
+    for (i = 1; i < argc;) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/cli/cli_repack.cpp b/cli/cli_repack.cpp
index 918f54a830b0a347c8e35d650360fc05dc712ba3..a92723d60ddaa59af5c43c0adf2bd0db77ac2f85 100644 (file)
--- a/cli/cli_repack.cpp
+++ b/cli/cli_repack.cpp
@@ -80,7 +80,7 @@ command(int argc, char *argv[])
 {
     int i;
 
-    for (i = 0; i < argc; ++i) {
+    for (i = 1; i < argc; ++i) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/cli/cli_trace.cpp b/cli/cli_trace.cpp
index 92251b4bef0ad6cd10b4d9b924a0414f27b9e67b..68526cb4b01c69a5ce559f93a2227434d20cfe90 100644 (file)
--- a/cli/cli_trace.cpp
+++ b/cli/cli_trace.cpp
@@ -64,7 +64,7 @@ command(int argc, char *argv[])
     const char *output = NULL;
     int i;
 
-    for (i = 0; i < argc; ) {
+    for (i = 1; i < argc; ) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/cli/cli_trim.cpp b/cli/cli_trim.cpp
index 8fdd93c3153f2ace0c21870a78da5a3c6daed012..515b4b61b8a9068ca50241900fd90facbe7a1020 100644 (file)
--- a/cli/cli_trim.cpp
+++ b/cli/cli_trim.cpp
@@ -56,7 +56,7 @@ command(int argc, char *argv[])
     trace::CallSet calls(trace::FREQUENCY_ALL);
     int i;
 
-    for (i = 0; i < argc;) {
+    for (i = 1; i < argc;) {
         const char *arg = argv[i];
 
         if (arg[0] != '-') {

diff --git a/scripts/tracediff.sh b/scripts/tracediff.sh
index 9f9633d2d4666f94e41411d427402279a59ca62d..819a25120ebdfceb2a8234ce1addd3923f1b0916 100755 (executable)
--- a/scripts/tracediff.sh
+++ b/scripts/tracediff.sh
@@ -31,7 +31,7 @@ APITRACE=${APITRACE:-apitrace}
 $APITRACE dump
 
 stripdump () {
-    $APITRACE dump --color=never --no-arg-names "$1" \
+    $APITRACE dump --color=never --arg-names=no "$1" \
     | sed \
         -e 's/\r$//g' \
         -e 's/^[0-9]\+ //' \