Make Args::Table::Terminator a constexpr

LLNL · Aug 2, 2024 · 1f19782 · 1f19782
1 parent 319d8c7
commit 1f19782
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 10 deletions.
diff --git a/include/caliper/tools-util/Args.h b/include/caliper/tools-util/Args.h
@@ -32,11 +32,11 @@ class Args {
         bool        has_argument;
 
         const char* info;
-        const char* argument_info;
-
-        static const Table Terminator;
+        const char* argument_info;      
     };
 
+    static constexpr Table Terminator = { nullptr, nullptr, 0, false, nullptr, nullptr };
+
     Args();
 
     Args(const Table table[]);

diff --git a/src/tools/cali-query/cali-query.cpp b/src/tools/cali-query/cali-query.cpp
@@ -121,7 +121,7 @@ namespace
           "List global run metadata. Use with -j, -t, etc. to select output format.",
           nullptr
         },
-        Args::Table::Terminator
+        Args::Terminator
     };
 
     /// A node record filter that filters redundant identical node records.

diff --git a/src/tools/cali-stat/cali-stat.cpp b/src/tools/cali-stat/cali-stat.cpp
@@ -38,7 +38,7 @@ namespace
         },
         { "output", "output", 'o', true,  "Set the output file name", "FILE"  },
         { "help",   "help",   'h', false, "Print help message",       nullptr },
-        Args::Table::Terminator
+        Args::Terminator
     };
 
     class ReuseStat {

diff --git a/src/tools/mpi-caliquery/mpi-caliquery.cpp b/src/tools/mpi-caliquery/mpi-caliquery.cpp
@@ -99,7 +99,7 @@ const Args::Table option_table[] = {
     { "verbose", "verbose", 'v', false, "Be verbose.",              nullptr },
     { "help",    "help",    'h', true,  "Print help message",       nullptr },
     { "output", "output",   'o', true,  "Set the output file name", "FILE"  },
-    Args::Table::Terminator
+    Args::Terminator
 };
 
 

diff --git a/src/tools/util/Args.cpp b/src/tools/util/Args.cpp
@@ -12,8 +12,6 @@
 using namespace std;
 using namespace util;
 
-const Args::Table Args::Table::Terminator { 0, 0, 0, false, 0, 0 };
-
 struct Args::ArgsImpl
 {
     vector<Args::Table> m_options;     ///< Option list

diff --git a/test/cali-annotation-perftest.cpp b/test/cali-annotation-perftest.cpp
@@ -166,7 +166,7 @@ int main(int argc, char* argv[])
         { "quiet", "quiet", 'q', false, "Don't print output", nullptr },
         { "help",  "help",  'h', false, "Print help",         nullptr },
 
-        util::Args::Table::Terminator
+        util::Args::Terminator
     };
 
     // --- initialization

diff --git a/test/cali-flush-perftest.cpp b/test/cali-flush-perftest.cpp
@@ -79,7 +79,7 @@ int main(int argc, char* argv[])
 
         { "help", "help", 'h', false, "Print help", nullptr },
 
-        util::Args::Table::Terminator
+        util::Args::Terminator
     };
 
     // --- Initialization