Revision: eliminate unused 'iterations' and conform simd_test to the usual standard used elsewhere

Signed-off-by: Larry Gritz <[email protected]>

Author: lgritz

src/libOpenImageIO/image_span_test.cpp

@@ -19,7 +19,6 @@
 using namespace OIIO;
 
 
-static int iterations = 0;  // 1000000;
 static int ntrials    = 5;
 
 
@@ -562,9 +561,9 @@ getargs(int argc, char* argv[])
           "image_span_test -- unit test and benchmarks for OpenImageIO/image_span.h\n" OIIO_INTRO_STRING)
         .usage("image_span_test [options]");
 
-    ap.arg("--iterations %d", &iterations)
-        .help(Strutil::fmt::format("Number of iterations (default: {})",
-                                   iterations));
+    // ap.arg("--iterations %d", &iterations)
+    //     .help(Strutil::fmt::format("Number of iterations (default: {})",
+    //                                iterations));
     ap.arg("--trials %d", &ntrials).help("Number of trials");
 
     ap.parse_args(argc, (const char**)argv);

src/libutil/simd_test.cpp

@@ -43,8 +43,10 @@ getargs(int argc, char* argv[])
              OIIO_INTRO_STRING)
       .usage("simd_test [options]");
 
-    ap.arg("--iterations %d", &iterations)
+    ap.arg("--iters %d", &iterations)
       .help(Strutil::fmt::format("Number of iterations (default: {})", iterations));
+    ap.arg("--iterations %d", &iterations)
+      .hidden();  // obsolete synonym for --iters
     ap.arg("--trials %d", &ntrials)
       .help("Number of trials");