From bd68a052f292b5df7c5717bd880d796ac7507fc0 Mon Sep 17 00:00:00 2001 From: Raphael Isemann Date: Sun, 28 Jul 2019 06:24:07 +0000 Subject: [lldb] Also include the array definition in CommandOptions.inc Summary: Right now our CommandOptions.inc only generates the initializer for the options list but not the array declaration boilerplate around it. As the array definition is identical for all arrays, we might as well also let the CommandOptions.inc generate it alongside the initializers. This patch will also allow us to generate additional declarations related to that option list in the future (e.g. a enum class representing the specific options which would make our handling code less prone). This patch also fixes a few option tables that didn't follow our naming style. Reviewers: JDevlieghere Reviewed By: JDevlieghere Subscribers: abidh, lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D65331 llvm-svn: 367186 --- lldb/source/Commands/CommandObjectLog.cpp | 2 -- 1 file changed, 2 deletions(-) (limited to 'lldb/source/Commands/CommandObjectLog.cpp') diff --git a/lldb/source/Commands/CommandObjectLog.cpp b/lldb/source/Commands/CommandObjectLog.cpp index e096836..2a591d2 100644 --- a/lldb/source/Commands/CommandObjectLog.cpp +++ b/lldb/source/Commands/CommandObjectLog.cpp @@ -31,10 +31,8 @@ using namespace lldb; using namespace lldb_private; -static constexpr OptionDefinition g_log_options[] = { #define LLDB_OPTIONS_log #include "CommandOptions.inc" -}; class CommandObjectLogEnable : public CommandObjectParsed { public: -- cgit v1.1