Refactoring: replace a bunch of static array size computation or hardcoded constant
with a template function 'arraysize(static_array)', defined in Utils.h.
git-svn-id: https://llvm.org/svn/llvm-project/llvdb/trunk@139444 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/source/Interpreter/OptionGroupArchitecture.cpp b/source/Interpreter/OptionGroupArchitecture.cpp
index 9446f2e..377bac0 100644
--- a/source/Interpreter/OptionGroupArchitecture.cpp
+++ b/source/Interpreter/OptionGroupArchitecture.cpp
@@ -13,6 +13,7 @@
// C++ Includes
// Other libraries and framework includes
// Project includes
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -29,15 +30,13 @@
static OptionDefinition
g_option_table[] =
{
-{ LLDB_OPT_SET_1 , false, "arch" , 'a', required_argument, NULL, 0, eArgTypeArchitecture , "Specify the architecture for the target."},
+ { LLDB_OPT_SET_1 , false, "arch" , 'a', required_argument, NULL, 0, eArgTypeArchitecture , "Specify the architecture for the target."},
};
-const uint32_t k_num_file_options = sizeof(g_option_table)/sizeof(OptionDefinition);
-
uint32_t
OptionGroupArchitecture::GetNumDefinitions ()
{
- return k_num_file_options;
+ return arraysize(g_option_table);
}
const OptionDefinition *
diff --git a/source/Interpreter/OptionGroupFormat.cpp b/source/Interpreter/OptionGroupFormat.cpp
index a42099d..7f6ebbf 100644
--- a/source/Interpreter/OptionGroupFormat.cpp
+++ b/source/Interpreter/OptionGroupFormat.cpp
@@ -13,6 +13,7 @@
// C++ Includes
// Other libraries and framework includes
// Project includes
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -35,14 +36,13 @@
static OptionDefinition
g_option_table[] =
{
-{ LLDB_OPT_SET_1 , false, "format", 'f', required_argument, NULL, 0, eArgTypeFormat , "Specify a format to be used for display."},
+ { LLDB_OPT_SET_1 , false, "format", 'f', required_argument, NULL, 0, eArgTypeFormat , "Specify a format to be used for display."},
};
-const uint32_t k_num_file_options = sizeof(g_option_table)/sizeof(OptionDefinition);
uint32_t
OptionGroupFormat::GetNumDefinitions ()
{
- return k_num_file_options;
+ return arraysize(g_option_table);
}
const OptionDefinition *
diff --git a/source/Interpreter/OptionGroupOutputFile.cpp b/source/Interpreter/OptionGroupOutputFile.cpp
index 541c8b4..f5daa3a 100644
--- a/source/Interpreter/OptionGroupOutputFile.cpp
+++ b/source/Interpreter/OptionGroupOutputFile.cpp
@@ -13,6 +13,7 @@
// C++ Includes
// Other libraries and framework includes
// Project includes
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -30,16 +31,14 @@
static OptionDefinition
g_option_table[] =
{
-{ LLDB_OPT_SET_1 , false, "outfile", 'o', required_argument, NULL, 0, eArgTypePath , "Specify a path for capturing command output."},
-{ LLDB_OPT_SET_1 , false, "append-outfile" , 'A', no_argument, NULL, 0, eArgTypeNone , "Append to the the file specified with '--outfile <path>'."},
+ { LLDB_OPT_SET_1 , false, "outfile", 'o', required_argument, NULL, 0, eArgTypePath , "Specify a path for capturing command output."},
+ { LLDB_OPT_SET_1 , false, "append-outfile" , 'A', no_argument, NULL, 0, eArgTypeNone , "Append to the the file specified with '--outfile <path>'."},
};
-const uint32_t k_num_file_options = sizeof(g_option_table)/sizeof(OptionDefinition);
-
uint32_t
OptionGroupOutputFile::GetNumDefinitions ()
{
- return k_num_file_options;
+ return arraysize(g_option_table);
}
const OptionDefinition *
diff --git a/source/Interpreter/OptionGroupPlatform.cpp b/source/Interpreter/OptionGroupPlatform.cpp
index 1734a79..910a6e9 100644
--- a/source/Interpreter/OptionGroupPlatform.cpp
+++ b/source/Interpreter/OptionGroupPlatform.cpp
@@ -15,6 +15,7 @@
// Project includes
#include "lldb/Interpreter/CommandInterpreter.h"
#include "lldb/Target/Platform.h"
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -67,8 +68,6 @@
{ LLDB_OPT_SET_ALL, false, "sysroot" , 's', required_argument, NULL, 0, eArgTypeFilename, "Specify the SDK root directory that contains a root of all remote system files." }
};
-static const uint32_t k_option_table_size = sizeof(g_option_table)/sizeof (OptionDefinition);
-
const OptionDefinition*
OptionGroupPlatform::GetDefinitions ()
{
@@ -81,8 +80,8 @@
OptionGroupPlatform::GetNumDefinitions ()
{
if (m_include_platform_option)
- return k_option_table_size;
- return k_option_table_size - 1;
+ return arraysize(g_option_table);
+ return arraysize(g_option_table) - 1;
}
diff --git a/source/Interpreter/OptionGroupUUID.cpp b/source/Interpreter/OptionGroupUUID.cpp
index b1b3c77..d009008 100644
--- a/source/Interpreter/OptionGroupUUID.cpp
+++ b/source/Interpreter/OptionGroupUUID.cpp
@@ -13,6 +13,7 @@
// C++ Includes
// Other libraries and framework includes
// Project includes
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -29,15 +30,13 @@
static OptionDefinition
g_option_table[] =
{
-{ LLDB_OPT_SET_1 , false, "uuid", 'u', required_argument, NULL, 0, eArgTypeNone, "A module UUID value."},
+ { LLDB_OPT_SET_1 , false, "uuid", 'u', required_argument, NULL, 0, eArgTypeNone, "A module UUID value."},
};
-const uint32_t k_num_file_options = sizeof(g_option_table)/sizeof(OptionDefinition);
-
uint32_t
OptionGroupUUID::GetNumDefinitions ()
{
- return k_num_file_options;
+ return arraysize(g_option_table);
}
const OptionDefinition *
diff --git a/source/Interpreter/OptionGroupValueObjectDisplay.cpp b/source/Interpreter/OptionGroupValueObjectDisplay.cpp
index 5f540d6..4557883 100644
--- a/source/Interpreter/OptionGroupValueObjectDisplay.cpp
+++ b/source/Interpreter/OptionGroupValueObjectDisplay.cpp
@@ -15,6 +15,7 @@
// Project includes
#include "lldb/Target/Target.h"
#include "lldb/Interpreter/CommandInterpreter.h"
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -45,12 +46,10 @@
{ 0, false, NULL, 0, 0, NULL, NULL, eArgTypeNone, NULL }
};
-const uint32_t k_num_file_options = sizeof(g_option_table)/sizeof(OptionDefinition);
-
uint32_t
OptionGroupValueObjectDisplay::GetNumDefinitions ()
{
- return k_num_file_options;
+ return arraysize(g_option_table);
}
const OptionDefinition *
diff --git a/source/Interpreter/OptionGroupWatchpoint.cpp b/source/Interpreter/OptionGroupWatchpoint.cpp
index 6d1fcb7..be1bf57 100644
--- a/source/Interpreter/OptionGroupWatchpoint.cpp
+++ b/source/Interpreter/OptionGroupWatchpoint.cpp
@@ -15,6 +15,7 @@
// Project includes
#include "lldb/lldb-enumerations.h"
#include "lldb/Interpreter/Args.h"
+#include "lldb/Utility/Utils.h"
using namespace lldb;
using namespace lldb_private;
@@ -84,7 +85,5 @@
uint32_t
OptionGroupWatchpoint::GetNumDefinitions ()
{
- return 1;
+ return arraysize(g_option_table);
}
-
-