diff --git a/lldb/.clang-tidy b/lldb/.clang-tidy --- a/lldb/.clang-tidy +++ b/lldb/.clang-tidy @@ -1,4 +1,4 @@ -Checks: '-readability-identifier-naming,modernize-use-default-member-init' +Checks: '-readability-identifier-naming,modernize-use-default-member-init,modernize-use-equals-default' InheritParentConfig: true CheckOptions: - key: modernize-use-default-member-init.IgnoreMacros diff --git a/lldb/include/lldb/Core/EmulateInstruction.h b/lldb/include/lldb/Core/EmulateInstruction.h --- a/lldb/include/lldb/Core/EmulateInstruction.h +++ b/lldb/include/lldb/Core/EmulateInstruction.h @@ -237,7 +237,7 @@ uint32_t isa; } info; - Context() {} + Context() = default; void SetRegisterPlusOffset(RegisterInfo base_reg, int64_t signed_offset) { info_type = eInfoTypeRegisterPlusOffset; diff --git a/lldb/include/lldb/Core/Opcode.h b/lldb/include/lldb/Core/Opcode.h --- a/lldb/include/lldb/Core/Opcode.h +++ b/lldb/include/lldb/Core/Opcode.h @@ -38,7 +38,7 @@ eTypeBytes }; - Opcode() {} + Opcode() = default; Opcode(uint8_t inst, lldb::ByteOrder order) : m_byte_order(order), m_type(eType8) { diff --git a/lldb/include/lldb/Core/StreamBuffer.h b/lldb/include/lldb/Core/StreamBuffer.h --- a/lldb/include/lldb/Core/StreamBuffer.h +++ b/lldb/include/lldb/Core/StreamBuffer.h @@ -23,7 +23,7 @@ StreamBuffer(uint32_t flags, uint32_t addr_size, lldb::ByteOrder byte_order) : Stream(flags, addr_size, byte_order), m_packet() {} - ~StreamBuffer() override {} + ~StreamBuffer() override = default; void Flush() override { // Nothing to do when flushing a buffer based stream... diff --git a/lldb/include/lldb/Core/ThreadSafeValue.h b/lldb/include/lldb/Core/ThreadSafeValue.h --- a/lldb/include/lldb/Core/ThreadSafeValue.h +++ b/lldb/include/lldb/Core/ThreadSafeValue.h @@ -17,12 +17,10 @@ template class ThreadSafeValue { public: - // Constructors and Destructors - ThreadSafeValue() : m_value(), m_mutex() {} - + ThreadSafeValue() = default; ThreadSafeValue(const T &value) : m_value(value), m_mutex() {} - ~ThreadSafeValue() {} + ~ThreadSafeValue() = default; T GetValue() const { T value; diff --git a/lldb/include/lldb/Core/UserSettingsController.h b/lldb/include/lldb/Core/UserSettingsController.h --- a/lldb/include/lldb/Core/UserSettingsController.h +++ b/lldb/include/lldb/Core/UserSettingsController.h @@ -32,12 +32,12 @@ class Properties { public: - Properties() : m_collection_sp() {} + Properties() = default; Properties(const lldb::OptionValuePropertiesSP &collection_sp) : m_collection_sp(collection_sp) {} - virtual ~Properties() {} + virtual ~Properties() = default; virtual lldb::OptionValuePropertiesSP GetValueProperties() const { // This function is virtual in case subclasses want to lazily implement diff --git a/lldb/include/lldb/DataFormatters/DumpValueObjectOptions.h b/lldb/include/lldb/DataFormatters/DumpValueObjectOptions.h --- a/lldb/include/lldb/DataFormatters/DumpValueObjectOptions.h +++ b/lldb/include/lldb/DataFormatters/DumpValueObjectOptions.h @@ -39,7 +39,7 @@ size_t m_base_element = 0; size_t m_stride = 0; - PointerAsArraySettings() {} + PointerAsArraySettings() = default; PointerAsArraySettings(size_t elem_count, size_t base_elem = 0, size_t stride = 1) diff --git a/lldb/include/lldb/DataFormatters/TypeSummary.h b/lldb/include/lldb/DataFormatters/TypeSummary.h --- a/lldb/include/lldb/DataFormatters/TypeSummary.h +++ b/lldb/include/lldb/DataFormatters/TypeSummary.h @@ -52,7 +52,7 @@ class Flags { public: - Flags() {} + Flags() = default; Flags(const Flags &other) : m_flags(other.m_flags) {} diff --git a/lldb/include/lldb/DataFormatters/TypeSynthetic.h b/lldb/include/lldb/DataFormatters/TypeSynthetic.h --- a/lldb/include/lldb/DataFormatters/TypeSynthetic.h +++ b/lldb/include/lldb/DataFormatters/TypeSynthetic.h @@ -133,7 +133,7 @@ public: class Flags { public: - Flags() {} + Flags() = default; Flags(const Flags &other) : m_flags(other.m_flags) {} diff --git a/lldb/include/lldb/DataFormatters/ValueObjectPrinter.h b/lldb/include/lldb/DataFormatters/ValueObjectPrinter.h --- a/lldb/include/lldb/DataFormatters/ValueObjectPrinter.h +++ b/lldb/include/lldb/DataFormatters/ValueObjectPrinter.h @@ -27,7 +27,7 @@ ValueObjectPrinter(ValueObject *valobj, Stream *s, const DumpValueObjectOptions &options); - ~ValueObjectPrinter() {} + ~ValueObjectPrinter() = default; bool PrintValueObject(); diff --git a/lldb/include/lldb/Expression/Expression.h b/lldb/include/lldb/Expression/Expression.h --- a/lldb/include/lldb/Expression/Expression.h +++ b/lldb/include/lldb/Expression/Expression.h @@ -39,7 +39,7 @@ Expression(ExecutionContextScope &exe_scope); /// Destructor - virtual ~Expression() {} + virtual ~Expression() = default; /// Return the string that the parser should parse. Must be a full /// translation unit. diff --git a/lldb/include/lldb/Expression/ExpressionParser.h b/lldb/include/lldb/Expression/ExpressionParser.h --- a/lldb/include/lldb/Expression/ExpressionParser.h +++ b/lldb/include/lldb/Expression/ExpressionParser.h @@ -41,7 +41,7 @@ : m_expr(expr), m_generate_debug_info(generate_debug_info) {} /// Destructor - virtual ~ExpressionParser(){}; + virtual ~ExpressionParser() = default; /// Attempts to find possible command line completions for the given /// expression. diff --git a/lldb/include/lldb/Expression/ExpressionTypeSystemHelper.h b/lldb/include/lldb/Expression/ExpressionTypeSystemHelper.h --- a/lldb/include/lldb/Expression/ExpressionTypeSystemHelper.h +++ b/lldb/include/lldb/Expression/ExpressionTypeSystemHelper.h @@ -36,7 +36,7 @@ ExpressionTypeSystemHelper(LLVMCastKind kind) : m_kind(kind) {} - ~ExpressionTypeSystemHelper() {} + ~ExpressionTypeSystemHelper() = default; protected: LLVMCastKind m_kind; diff --git a/lldb/include/lldb/Expression/Materializer.h b/lldb/include/lldb/Expression/Materializer.h --- a/lldb/include/lldb/Expression/Materializer.h +++ b/lldb/include/lldb/Expression/Materializer.h @@ -90,7 +90,7 @@ class Entity { public: - Entity() {} + Entity() = default; virtual ~Entity() = default; diff --git a/lldb/include/lldb/Host/FileCache.h b/lldb/include/lldb/Host/FileCache.h --- a/lldb/include/lldb/Host/FileCache.h +++ b/lldb/include/lldb/Host/FileCache.h @@ -21,7 +21,7 @@ namespace lldb_private { class FileCache { private: - FileCache() {} + FileCache() = default; typedef std::map FDToFileMap; diff --git a/lldb/include/lldb/Host/HostInfoBase.h b/lldb/include/lldb/Host/HostInfoBase.h --- a/lldb/include/lldb/Host/HostInfoBase.h +++ b/lldb/include/lldb/Host/HostInfoBase.h @@ -33,8 +33,8 @@ class HostInfoBase { private: // Static class, unconstructable. - HostInfoBase() {} - ~HostInfoBase() {} + HostInfoBase() = default; + ~HostInfoBase() = default; public: /// A helper function for determining the liblldb location. It receives a diff --git a/lldb/include/lldb/Host/HostNativeProcessBase.h b/lldb/include/lldb/Host/HostNativeProcessBase.h --- a/lldb/include/lldb/Host/HostNativeProcessBase.h +++ b/lldb/include/lldb/Host/HostNativeProcessBase.h @@ -27,7 +27,7 @@ HostNativeProcessBase() : m_process(LLDB_INVALID_PROCESS) {} explicit HostNativeProcessBase(lldb::process_t process) : m_process(process) {} - virtual ~HostNativeProcessBase() {} + virtual ~HostNativeProcessBase() = default; virtual Status Terminate() = 0; virtual Status GetMainModule(FileSpec &file_spec) const = 0; diff --git a/lldb/include/lldb/Host/HostNativeThreadBase.h b/lldb/include/lldb/Host/HostNativeThreadBase.h --- a/lldb/include/lldb/Host/HostNativeThreadBase.h +++ b/lldb/include/lldb/Host/HostNativeThreadBase.h @@ -27,9 +27,9 @@ const HostNativeThreadBase &operator=(const HostNativeThreadBase &) = delete; public: - HostNativeThreadBase(); + HostNativeThreadBase() = default; explicit HostNativeThreadBase(lldb::thread_t thread); - virtual ~HostNativeThreadBase() {} + virtual ~HostNativeThreadBase() = default; virtual Status Join(lldb::thread_result_t *result) = 0; virtual Status Cancel() = 0; @@ -45,7 +45,7 @@ static lldb::thread_result_t THREAD_ROUTINE ThreadCreateTrampoline(lldb::thread_arg_t arg); - lldb::thread_t m_thread; + lldb::thread_t m_thread = LLDB_INVALID_HOST_THREAD; lldb::thread_result_t m_result = 0; }; } diff --git a/lldb/include/lldb/Host/MainLoopBase.h b/lldb/include/lldb/Host/MainLoopBase.h --- a/lldb/include/lldb/Host/MainLoopBase.h +++ b/lldb/include/lldb/Host/MainLoopBase.h @@ -33,8 +33,8 @@ class ReadHandle; public: - MainLoopBase() {} - virtual ~MainLoopBase() {} + MainLoopBase() = default; + virtual ~MainLoopBase() = default; typedef std::unique_ptr ReadHandleUP; diff --git a/lldb/include/lldb/Host/ProcessLauncher.h b/lldb/include/lldb/Host/ProcessLauncher.h --- a/lldb/include/lldb/Host/ProcessLauncher.h +++ b/lldb/include/lldb/Host/ProcessLauncher.h @@ -17,7 +17,7 @@ class ProcessLauncher { public: - virtual ~ProcessLauncher() {} + virtual ~ProcessLauncher() = default; virtual HostProcess LaunchProcess(const ProcessLaunchInfo &launch_info, Status &error) = 0; }; diff --git a/lldb/include/lldb/Host/ProcessRunLock.h b/lldb/include/lldb/Host/ProcessRunLock.h --- a/lldb/include/lldb/Host/ProcessRunLock.h +++ b/lldb/include/lldb/Host/ProcessRunLock.h @@ -35,7 +35,7 @@ class ProcessRunLocker { public: - ProcessRunLocker() {} + ProcessRunLocker() = default; ~ProcessRunLocker() { Unlock(); } diff --git a/lldb/include/lldb/Host/Terminal.h b/lldb/include/lldb/Host/Terminal.h --- a/lldb/include/lldb/Host/Terminal.h +++ b/lldb/include/lldb/Host/Terminal.h @@ -21,7 +21,7 @@ public: Terminal(int fd = -1) : m_fd(fd) {} - ~Terminal() {} + ~Terminal() = default; bool IsATerminal() const; diff --git a/lldb/include/lldb/Host/common/NativeProcessProtocol.h b/lldb/include/lldb/Host/common/NativeProcessProtocol.h --- a/lldb/include/lldb/Host/common/NativeProcessProtocol.h +++ b/lldb/include/lldb/Host/common/NativeProcessProtocol.h @@ -46,7 +46,7 @@ // NativeProcessProtocol class NativeProcessProtocol { public: - virtual ~NativeProcessProtocol() {} + virtual ~NativeProcessProtocol() = default; virtual Status Resume(const ResumeActionList &resume_actions) = 0; @@ -217,7 +217,7 @@ // Callbacks for low-level process state changes class NativeDelegate { public: - virtual ~NativeDelegate() {} + virtual ~NativeDelegate() = default; virtual void InitializeDelegate(NativeProcessProtocol *process) = 0; diff --git a/lldb/include/lldb/Host/common/NativeThreadProtocol.h b/lldb/include/lldb/Host/common/NativeThreadProtocol.h --- a/lldb/include/lldb/Host/common/NativeThreadProtocol.h +++ b/lldb/include/lldb/Host/common/NativeThreadProtocol.h @@ -21,7 +21,7 @@ public: NativeThreadProtocol(NativeProcessProtocol &process, lldb::tid_t tid); - virtual ~NativeThreadProtocol() {} + virtual ~NativeThreadProtocol() = default; virtual std::string GetName() = 0; diff --git a/lldb/include/lldb/Interpreter/CommandInterpreter.h b/lldb/include/lldb/Interpreter/CommandInterpreter.h --- a/lldb/include/lldb/Interpreter/CommandInterpreter.h +++ b/lldb/include/lldb/Interpreter/CommandInterpreter.h @@ -33,7 +33,7 @@ class CommandInterpreterRunResult { public: - CommandInterpreterRunResult() {} + CommandInterpreterRunResult() = default; uint32_t GetNumErrors() const { return m_num_errors; } @@ -100,9 +100,7 @@ m_echo_comment_commands(echo_comments), m_print_results(print_results), m_print_errors(print_errors), m_add_to_history(add_to_history) {} - CommandInterpreterRunOptions() - - {} + CommandInterpreterRunOptions() = default; void SetSilent(bool silent) { LazyBool value = silent ? eLazyBoolNo : eLazyBoolYes; diff --git a/lldb/include/lldb/Interpreter/OptionValue.h b/lldb/include/lldb/Interpreter/OptionValue.h --- a/lldb/include/lldb/Interpreter/OptionValue.h +++ b/lldb/include/lldb/Interpreter/OptionValue.h @@ -60,7 +60,7 @@ eDumpGroupExport = (eDumpOptionCommand | eDumpOptionName | eDumpOptionValue) }; - OptionValue() {} + OptionValue() = default; virtual ~OptionValue() = default; diff --git a/lldb/include/lldb/Interpreter/Options.h b/lldb/include/lldb/Interpreter/Options.h --- a/lldb/include/lldb/Interpreter/Options.h +++ b/lldb/include/lldb/Interpreter/Options.h @@ -254,7 +254,7 @@ class OptionGroupOptions : public Options { public: - OptionGroupOptions() {} + OptionGroupOptions() = default; ~OptionGroupOptions() override = default; diff --git a/lldb/include/lldb/Interpreter/ScriptInterpreter.h b/lldb/include/lldb/Interpreter/ScriptInterpreter.h --- a/lldb/include/lldb/Interpreter/ScriptInterpreter.h +++ b/lldb/include/lldb/Interpreter/ScriptInterpreter.h @@ -37,6 +37,40 @@ operator=(const ScriptInterpreterLocker &) = delete; }; +class ExecuteScriptOptions { +public: + ExecuteScriptOptions() = default; + + bool GetEnableIO() const { return m_enable_io; } + + bool GetSetLLDBGlobals() const { return m_set_lldb_globals; } + + // If this is true then any exceptions raised by the script will be + // cleared with PyErr_Clear(). If false then they will be left for + // the caller to clean up + bool GetMaskoutErrors() const { return m_maskout_errors; } + + ExecuteScriptOptions &SetEnableIO(bool enable) { + m_enable_io = enable; + return *this; + } + + ExecuteScriptOptions &SetSetLLDBGlobals(bool set) { + m_set_lldb_globals = set; + return *this; + } + + ExecuteScriptOptions &SetMaskoutErrors(bool maskout) { + m_maskout_errors = maskout; + return *this; + } + +private: + bool m_enable_io = true; + bool m_set_lldb_globals = true; + bool m_maskout_errors = true; +}; + class ScriptInterpreterIORedirect { public: /// Create an IO redirect. If IO is enabled, this will redirects the output @@ -92,40 +126,6 @@ ~ScriptInterpreter() override = default; - struct ExecuteScriptOptions { - public: - ExecuteScriptOptions() {} - - bool GetEnableIO() const { return m_enable_io; } - - bool GetSetLLDBGlobals() const { return m_set_lldb_globals; } - - // If this is true then any exceptions raised by the script will be - // cleared with PyErr_Clear(). If false then they will be left for - // the caller to clean up - bool GetMaskoutErrors() const { return m_maskout_errors; } - - ExecuteScriptOptions &SetEnableIO(bool enable) { - m_enable_io = enable; - return *this; - } - - ExecuteScriptOptions &SetSetLLDBGlobals(bool set) { - m_set_lldb_globals = set; - return *this; - } - - ExecuteScriptOptions &SetMaskoutErrors(bool maskout) { - m_maskout_errors = maskout; - return *this; - } - - private: - bool m_enable_io = true; - bool m_set_lldb_globals = true; - bool m_maskout_errors = true; - }; - virtual bool Interrupt() { return false; } virtual bool ExecuteOneLine( diff --git a/lldb/include/lldb/Symbol/CompactUnwindInfo.h b/lldb/include/lldb/Symbol/CompactUnwindInfo.h --- a/lldb/include/lldb/Symbol/CompactUnwindInfo.h +++ b/lldb/include/lldb/Symbol/CompactUnwindInfo.h @@ -63,7 +63,7 @@ // provides the upper bound of // function addresses that are described - UnwindIndex() {} + UnwindIndex() = default; bool operator<(const CompactUnwindInfo::UnwindIndex &rhs) const { return function_offset < rhs.function_offset; @@ -96,7 +96,7 @@ uint32_t personality_array_offset = 0; uint32_t personality_array_count = 0; - UnwindHeader() {} + UnwindHeader() = default; }; void ScanIndex(const lldb::ProcessSP &process_sp); diff --git a/lldb/include/lldb/Symbol/DeclVendor.h b/lldb/include/lldb/Symbol/DeclVendor.h --- a/lldb/include/lldb/Symbol/DeclVendor.h +++ b/lldb/include/lldb/Symbol/DeclVendor.h @@ -28,7 +28,7 @@ // Constructors and Destructors DeclVendor(DeclVendorKind kind) : m_kind(kind) {} - virtual ~DeclVendor() {} + virtual ~DeclVendor() = default; DeclVendorKind GetKind() const { return m_kind; } diff --git a/lldb/include/lldb/Symbol/Function.h b/lldb/include/lldb/Symbol/Function.h --- a/lldb/include/lldb/Symbol/Function.h +++ b/lldb/include/lldb/Symbol/Function.h @@ -271,7 +271,7 @@ class CallEdge { public: enum class AddrType : uint8_t { Call, AfterCall }; - virtual ~CallEdge() {} + virtual ~CallEdge() = default; /// Get the callee's definition. /// diff --git a/lldb/include/lldb/Symbol/ObjectFile.h b/lldb/include/lldb/Symbol/ObjectFile.h --- a/lldb/include/lldb/Symbol/ObjectFile.h +++ b/lldb/include/lldb/Symbol/ObjectFile.h @@ -25,9 +25,9 @@ class ObjectFileJITDelegate { public: - ObjectFileJITDelegate() {} + ObjectFileJITDelegate() = default; - virtual ~ObjectFileJITDelegate() {} + virtual ~ObjectFileJITDelegate() = default; virtual lldb::ByteOrder GetByteOrder() const = 0; diff --git a/lldb/include/lldb/Symbol/SymbolFile.h b/lldb/include/lldb/Symbol/SymbolFile.h --- a/lldb/include/lldb/Symbol/SymbolFile.h +++ b/lldb/include/lldb/Symbol/SymbolFile.h @@ -69,7 +69,7 @@ : m_objfile_sp(std::move(objfile_sp)), m_abilities(0), m_calculated_abilities(false) {} - ~SymbolFile() override {} + ~SymbolFile() override = default; /// Get a mask of what this symbol file supports for the object file /// that it was constructed with. diff --git a/lldb/include/lldb/Symbol/TaggedASTType.h b/lldb/include/lldb/Symbol/TaggedASTType.h --- a/lldb/include/lldb/Symbol/TaggedASTType.h +++ b/lldb/include/lldb/Symbol/TaggedASTType.h @@ -27,7 +27,7 @@ TaggedASTType() : CompilerType() {} - virtual ~TaggedASTType() {} + virtual ~TaggedASTType() = default; TaggedASTType &operator=(const TaggedASTType &tw) { CompilerType::operator=(tw); diff --git a/lldb/include/lldb/Symbol/Type.h b/lldb/include/lldb/Symbol/Type.h --- a/lldb/include/lldb/Symbol/Type.h +++ b/lldb/include/lldb/Symbol/Type.h @@ -51,7 +51,7 @@ SymbolFileType(SymbolFile &symbol_file, const lldb::TypeSP &type_sp); - ~SymbolFileType() {} + ~SymbolFileType() = default; Type *operator->() { return GetType(); } @@ -239,7 +239,7 @@ public: TypeImpl() = default; - ~TypeImpl() {} + ~TypeImpl() = default; TypeImpl(const lldb::TypeSP &type_sp); diff --git a/lldb/include/lldb/Target/ExecutionContextScope.h b/lldb/include/lldb/Target/ExecutionContextScope.h --- a/lldb/include/lldb/Target/ExecutionContextScope.h +++ b/lldb/include/lldb/Target/ExecutionContextScope.h @@ -31,7 +31,7 @@ /// context to allow functions that take a execution contexts to be called. class ExecutionContextScope { public: - virtual ~ExecutionContextScope() {} + virtual ~ExecutionContextScope() = default; virtual lldb::TargetSP CalculateTarget() = 0; diff --git a/lldb/include/lldb/Target/InstrumentationRuntimeStopInfo.h b/lldb/include/lldb/Target/InstrumentationRuntimeStopInfo.h --- a/lldb/include/lldb/Target/InstrumentationRuntimeStopInfo.h +++ b/lldb/include/lldb/Target/InstrumentationRuntimeStopInfo.h @@ -18,7 +18,7 @@ class InstrumentationRuntimeStopInfo : public StopInfo { public: - ~InstrumentationRuntimeStopInfo() override {} + ~InstrumentationRuntimeStopInfo() override = default; lldb::StopReason GetStopReason() const override { return lldb::eStopReasonInstrumentation; diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h --- a/lldb/include/lldb/Target/Process.h +++ b/lldb/include/lldb/Target/Process.h @@ -223,9 +223,7 @@ friend bool operator==(const ProcessModID &lhs, const ProcessModID &rhs); public: - ProcessModID() - - {} + ProcessModID() = default; ProcessModID(const ProcessModID &rhs) : m_stop_id(rhs.m_stop_id), m_memory_id(rhs.m_memory_id) {} diff --git a/lldb/include/lldb/Target/RegisterCheckpoint.h b/lldb/include/lldb/Target/RegisterCheckpoint.h --- a/lldb/include/lldb/Target/RegisterCheckpoint.h +++ b/lldb/include/lldb/Target/RegisterCheckpoint.h @@ -34,7 +34,7 @@ RegisterCheckpoint(Reason reason) : UserID(0), m_data_sp(), m_reason(reason) {} - ~RegisterCheckpoint() {} + ~RegisterCheckpoint() = default; lldb::DataBufferSP &GetData() { return m_data_sp; } diff --git a/lldb/include/lldb/Target/StackFrameRecognizer.h b/lldb/include/lldb/Target/StackFrameRecognizer.h --- a/lldb/include/lldb/Target/StackFrameRecognizer.h +++ b/lldb/include/lldb/Target/StackFrameRecognizer.h @@ -37,7 +37,7 @@ return lldb::ValueObjectSP(); } virtual lldb::StackFrameSP GetMostRelevantFrame() { return nullptr; }; - virtual ~RecognizedStackFrame(){}; + virtual ~RecognizedStackFrame() = default; std::string GetStopDescription() { return m_stop_desc; } @@ -63,7 +63,7 @@ return ""; } - virtual ~StackFrameRecognizer(){}; + virtual ~StackFrameRecognizer() = default; }; /// \class ScriptedStackFrameRecognizer @@ -80,7 +80,7 @@ public: ScriptedStackFrameRecognizer(lldb_private::ScriptInterpreter *interpreter, const char *pclass); - ~ScriptedStackFrameRecognizer() override {} + ~ScriptedStackFrameRecognizer() override = default; std::string GetName() override { return GetPythonClassName(); diff --git a/lldb/include/lldb/Target/StopInfo.h b/lldb/include/lldb/Target/StopInfo.h --- a/lldb/include/lldb/Target/StopInfo.h +++ b/lldb/include/lldb/Target/StopInfo.h @@ -25,7 +25,7 @@ // Constructors and Destructors StopInfo(Thread &thread, uint64_t value); - virtual ~StopInfo() {} + virtual ~StopInfo() = default; bool IsValid() const; diff --git a/lldb/include/lldb/Target/ThreadCollection.h b/lldb/include/lldb/Target/ThreadCollection.h --- a/lldb/include/lldb/Target/ThreadCollection.h +++ b/lldb/include/lldb/Target/ThreadCollection.h @@ -28,7 +28,7 @@ ThreadCollection(collection threads); - virtual ~ThreadCollection() {} + virtual ~ThreadCollection() = default; uint32_t GetSize(); diff --git a/lldb/include/lldb/Target/ThreadPlanStack.h b/lldb/include/lldb/Target/ThreadPlanStack.h --- a/lldb/include/lldb/Target/ThreadPlanStack.h +++ b/lldb/include/lldb/Target/ThreadPlanStack.h @@ -33,7 +33,7 @@ public: ThreadPlanStack(const Thread &thread, bool make_empty = false); - ~ThreadPlanStack() {} + ~ThreadPlanStack() = default; using PlanStack = std::vector; @@ -115,7 +115,7 @@ class ThreadPlanStackMap { public: ThreadPlanStackMap(Process &process) : m_process(process) {} - ~ThreadPlanStackMap() {} + ~ThreadPlanStackMap() = default; // Prune the map using the current_threads list. void Update(ThreadList ¤t_threads, bool delete_missing, diff --git a/lldb/include/lldb/Target/UnixSignals.h b/lldb/include/lldb/Target/UnixSignals.h --- a/lldb/include/lldb/Target/UnixSignals.h +++ b/lldb/include/lldb/Target/UnixSignals.h @@ -104,7 +104,7 @@ Signal(const char *name, bool default_suppress, bool default_stop, bool default_notify, const char *description, const char *alias); - ~Signal() {} + ~Signal() = default; }; virtual void Reset(); diff --git a/lldb/include/lldb/Target/Unwind.h b/lldb/include/lldb/Target/Unwind.h --- a/lldb/include/lldb/Target/Unwind.h +++ b/lldb/include/lldb/Target/Unwind.h @@ -21,7 +21,7 @@ Unwind(Thread &thread) : m_thread(thread), m_unwind_mutex() {} public: - virtual ~Unwind() {} + virtual ~Unwind() = default; void Clear() { std::lock_guard guard(m_unwind_mutex); diff --git a/lldb/include/lldb/Utility/Baton.h b/lldb/include/lldb/Utility/Baton.h --- a/lldb/include/lldb/Utility/Baton.h +++ b/lldb/include/lldb/Utility/Baton.h @@ -34,8 +34,8 @@ /// needed resources in their destructors. class Baton { public: - Baton() {} - virtual ~Baton() {} + Baton() = default; + virtual ~Baton() = default; virtual void *data() = 0; diff --git a/lldb/include/lldb/Utility/DataBuffer.h b/lldb/include/lldb/Utility/DataBuffer.h --- a/lldb/include/lldb/Utility/DataBuffer.h +++ b/lldb/include/lldb/Utility/DataBuffer.h @@ -48,7 +48,7 @@ /// and be downcast to the DataBuffer pure virtual interface. The virtual /// destructor ensures that destructing the base class will destruct the /// class that inherited from it correctly. - virtual ~DataBuffer() {} + virtual ~DataBuffer() = default; /// Get a pointer to the data. /// diff --git a/lldb/include/lldb/Utility/Event.h b/lldb/include/lldb/Utility/Event.h --- a/lldb/include/lldb/Utility/Event.h +++ b/lldb/include/lldb/Utility/Event.h @@ -101,7 +101,7 @@ public: EventDataReceipt() : EventData(), m_predicate(false) {} - ~EventDataReceipt() override {} + ~EventDataReceipt() override = default; static ConstString GetFlavorString() { static ConstString g_flavor("Process::ProcessEventData"); diff --git a/lldb/include/lldb/Utility/ReproducerInstrumentation.h b/lldb/include/lldb/Utility/ReproducerInstrumentation.h --- a/lldb/include/lldb/Utility/ReproducerInstrumentation.h +++ b/lldb/include/lldb/Utility/ReproducerInstrumentation.h @@ -469,7 +469,7 @@ /// The replayer interface. struct Replayer { - virtual ~Replayer() {} + virtual ~Replayer() = default; virtual void operator()(Deserializer &deserializer) const = 0; }; @@ -714,7 +714,7 @@ friend llvm::optional_detail::OptionalStorage; friend llvm::Optional; - InstrumentationData() {} + InstrumentationData() = default; InstrumentationData(Serializer &serializer, Registry ®istry) : m_serializer(&serializer), m_deserializer(nullptr), m_registry(®istry) {} diff --git a/lldb/include/lldb/Utility/StreamTee.h b/lldb/include/lldb/Utility/StreamTee.h --- a/lldb/include/lldb/Utility/StreamTee.h +++ b/lldb/include/lldb/Utility/StreamTee.h @@ -45,7 +45,7 @@ m_streams = rhs.m_streams; } - ~StreamTee() override {} + ~StreamTee() override = default; StreamTee &operator=(const StreamTee &rhs) { if (this != &rhs) { diff --git a/lldb/include/lldb/Utility/TraceGDBRemotePackets.h b/lldb/include/lldb/Utility/TraceGDBRemotePackets.h --- a/lldb/include/lldb/Utility/TraceGDBRemotePackets.h +++ b/lldb/include/lldb/Utility/TraceGDBRemotePackets.h @@ -60,7 +60,7 @@ /// jLLDBTraceStop gdb-remote packet /// \{ struct TraceStopRequest { - TraceStopRequest() {} + TraceStopRequest() = default; TraceStopRequest(llvm::StringRef type, const std::vector &tids); diff --git a/lldb/include/lldb/Utility/UserID.h b/lldb/include/lldb/Utility/UserID.h --- a/lldb/include/lldb/Utility/UserID.h +++ b/lldb/include/lldb/Utility/UserID.h @@ -33,7 +33,7 @@ UserID(lldb::user_id_t uid = LLDB_INVALID_UID) : m_uid(uid) {} /// Destructor. - ~UserID() {} + ~UserID() = default; /// Clears the object state. /// diff --git a/lldb/include/lldb/Utility/VMRange.h b/lldb/include/lldb/Utility/VMRange.h --- a/lldb/include/lldb/Utility/VMRange.h +++ b/lldb/include/lldb/Utility/VMRange.h @@ -26,13 +26,13 @@ typedef collection::iterator iterator; typedef collection::const_iterator const_iterator; - VMRange() {} + VMRange() = default; VMRange(lldb::addr_t start_addr, lldb::addr_t end_addr) : m_base_addr(start_addr), m_byte_size(end_addr > start_addr ? end_addr - start_addr : 0) {} - ~VMRange() {} + ~VMRange() = default; void Clear() { m_base_addr = 0; diff --git a/lldb/include/lldb/lldb-private-types.h b/lldb/include/lldb/lldb-private-types.h --- a/lldb/include/lldb/lldb-private-types.h +++ b/lldb/include/lldb/lldb-private-types.h @@ -100,7 +100,7 @@ using OptionEnumValues = llvm::ArrayRef; struct OptionValidator { - virtual ~OptionValidator() {} + virtual ~OptionValidator() = default; virtual bool IsValid(Platform &platform, const ExecutionContext &target) const = 0; virtual const char *ShortConditionString() const = 0; diff --git a/lldb/source/Breakpoint/BreakpointList.cpp b/lldb/source/Breakpoint/BreakpointList.cpp --- a/lldb/source/Breakpoint/BreakpointList.cpp +++ b/lldb/source/Breakpoint/BreakpointList.cpp @@ -26,7 +26,7 @@ : m_mutex(), m_breakpoints(), m_next_break_id(0), m_is_internal(is_internal) {} -BreakpointList::~BreakpointList() {} +BreakpointList::~BreakpointList() = default; break_id_t BreakpointList::Add(BreakpointSP &bp_sp, bool notify) { std::lock_guard guard(m_mutex); diff --git a/lldb/source/Breakpoint/BreakpointLocationCollection.cpp b/lldb/source/Breakpoint/BreakpointLocationCollection.cpp --- a/lldb/source/Breakpoint/BreakpointLocationCollection.cpp +++ b/lldb/source/Breakpoint/BreakpointLocationCollection.cpp @@ -21,7 +21,7 @@ : m_break_loc_collection(), m_collection_mutex() {} // Destructor -BreakpointLocationCollection::~BreakpointLocationCollection() {} +BreakpointLocationCollection::~BreakpointLocationCollection() = default; void BreakpointLocationCollection::Add(const BreakpointLocationSP &bp_loc) { std::lock_guard guard(m_collection_mutex); diff --git a/lldb/source/Breakpoint/BreakpointResolver.cpp b/lldb/source/Breakpoint/BreakpointResolver.cpp --- a/lldb/source/Breakpoint/BreakpointResolver.cpp +++ b/lldb/source/Breakpoint/BreakpointResolver.cpp @@ -65,7 +65,7 @@ lldb::addr_t offset) : m_breakpoint(bkpt), m_offset(offset), SubclassID(resolverTy) {} -BreakpointResolver::~BreakpointResolver() {} +BreakpointResolver::~BreakpointResolver() = default; BreakpointResolverSP BreakpointResolver::CreateFromStructuredData( const StructuredData::Dictionary &resolver_dict, Status &error) { diff --git a/lldb/source/Breakpoint/BreakpointSiteList.cpp b/lldb/source/Breakpoint/BreakpointSiteList.cpp --- a/lldb/source/Breakpoint/BreakpointSiteList.cpp +++ b/lldb/source/Breakpoint/BreakpointSiteList.cpp @@ -16,7 +16,7 @@ BreakpointSiteList::BreakpointSiteList() : m_mutex(), m_bp_site_list() {} -BreakpointSiteList::~BreakpointSiteList() {} +BreakpointSiteList::~BreakpointSiteList() = default; // Add breakpoint site to the list. However, if the element already exists in // the list, then we don't add it, and return LLDB_INVALID_BREAK_ID. diff --git a/lldb/source/Breakpoint/Stoppoint.cpp b/lldb/source/Breakpoint/Stoppoint.cpp --- a/lldb/source/Breakpoint/Stoppoint.cpp +++ b/lldb/source/Breakpoint/Stoppoint.cpp @@ -14,10 +14,10 @@ using namespace lldb_private; // Stoppoint constructor -Stoppoint::Stoppoint() {} +Stoppoint::Stoppoint() = default; // Destructor -Stoppoint::~Stoppoint() {} +Stoppoint::~Stoppoint() = default; break_id_t Stoppoint::GetID() const { return m_bid; } diff --git a/lldb/source/Breakpoint/WatchpointList.cpp b/lldb/source/Breakpoint/WatchpointList.cpp --- a/lldb/source/Breakpoint/WatchpointList.cpp +++ b/lldb/source/Breakpoint/WatchpointList.cpp @@ -14,7 +14,7 @@ WatchpointList::WatchpointList() : m_watchpoints(), m_mutex() {} -WatchpointList::~WatchpointList() {} +WatchpointList::~WatchpointList() = default; // Add a watchpoint to the list. lldb::watch_id_t WatchpointList::Add(const WatchpointSP &wp_sp, bool notify) { diff --git a/lldb/source/Commands/CommandObjectGUI.cpp b/lldb/source/Commands/CommandObjectGUI.cpp --- a/lldb/source/Commands/CommandObjectGUI.cpp +++ b/lldb/source/Commands/CommandObjectGUI.cpp @@ -22,7 +22,7 @@ : CommandObjectParsed(interpreter, "gui", "Switch into the curses based GUI mode.", "gui") {} -CommandObjectGUI::~CommandObjectGUI() {} +CommandObjectGUI::~CommandObjectGUI() = default; bool CommandObjectGUI::DoExecute(Args &args, CommandReturnObject &result) { #if LLDB_ENABLE_CURSES diff --git a/lldb/source/Commands/CommandObjectHelp.h b/lldb/source/Commands/CommandObjectHelp.h --- a/lldb/source/Commands/CommandObjectHelp.h +++ b/lldb/source/Commands/CommandObjectHelp.h @@ -34,7 +34,7 @@ public: CommandOptions() : Options() {} - ~CommandOptions() override {} + ~CommandOptions() override = default; Status SetOptionValue(uint32_t option_idx, llvm::StringRef option_arg, ExecutionContext *execution_context) override { diff --git a/lldb/source/Commands/CommandObjectLanguage.cpp b/lldb/source/Commands/CommandObjectLanguage.cpp --- a/lldb/source/Commands/CommandObjectLanguage.cpp +++ b/lldb/source/Commands/CommandObjectLanguage.cpp @@ -23,4 +23,4 @@ LanguageRuntime::InitializeCommands(this); } -CommandObjectLanguage::~CommandObjectLanguage() {} +CommandObjectLanguage::~CommandObjectLanguage() = default; diff --git a/lldb/source/Commands/CommandObjectPlatform.cpp b/lldb/source/Commands/CommandObjectPlatform.cpp --- a/lldb/source/Commands/CommandObjectPlatform.cpp +++ b/lldb/source/Commands/CommandObjectPlatform.cpp @@ -60,7 +60,7 @@ class OptionPermissions : public OptionGroup { public: - OptionPermissions() {} + OptionPermissions() = default; ~OptionPermissions() override = default; diff --git a/lldb/source/Commands/CommandObjectQuit.cpp b/lldb/source/Commands/CommandObjectQuit.cpp --- a/lldb/source/Commands/CommandObjectQuit.cpp +++ b/lldb/source/Commands/CommandObjectQuit.cpp @@ -22,7 +22,7 @@ : CommandObjectParsed(interpreter, "quit", "Quit the LLDB debugger.", "quit [exit-code]") {} -CommandObjectQuit::~CommandObjectQuit() {} +CommandObjectQuit::~CommandObjectQuit() = default; // returns true if there is at least one alive process is_a_detach will be true // if all alive processes will be detached when you quit and false if at least diff --git a/lldb/source/Commands/CommandObjectRegexCommand.cpp b/lldb/source/Commands/CommandObjectRegexCommand.cpp --- a/lldb/source/Commands/CommandObjectRegexCommand.cpp +++ b/lldb/source/Commands/CommandObjectRegexCommand.cpp @@ -23,7 +23,7 @@ m_entries(), m_is_removable(is_removable) {} // Destructor -CommandObjectRegexCommand::~CommandObjectRegexCommand() {} +CommandObjectRegexCommand::~CommandObjectRegexCommand() = default; bool CommandObjectRegexCommand::DoExecute(llvm::StringRef command, CommandReturnObject &result) { diff --git a/lldb/source/Commands/CommandObjectScript.cpp b/lldb/source/Commands/CommandObjectScript.cpp --- a/lldb/source/Commands/CommandObjectScript.cpp +++ b/lldb/source/Commands/CommandObjectScript.cpp @@ -84,7 +84,7 @@ "results. Start the interactive interpreter if no code is supplied.", "script [--language --] []") {} -CommandObjectScript::~CommandObjectScript() {} +CommandObjectScript::~CommandObjectScript() = default; bool CommandObjectScript::DoExecute(llvm::StringRef command, CommandReturnObject &result) { diff --git a/lldb/source/Commands/CommandObjectTarget.cpp b/lldb/source/Commands/CommandObjectTarget.cpp --- a/lldb/source/Commands/CommandObjectTarget.cpp +++ b/lldb/source/Commands/CommandObjectTarget.cpp @@ -158,9 +158,9 @@ class OptionGroupDependents : public OptionGroup { public: - OptionGroupDependents() {} + OptionGroupDependents() = default; - ~OptionGroupDependents() override {} + ~OptionGroupDependents() override = default; llvm::ArrayRef GetDefinitions() override { return llvm::makeArrayRef(g_target_dependents_options); diff --git a/lldb/source/Commands/CommandObjectVersion.cpp b/lldb/source/Commands/CommandObjectVersion.cpp --- a/lldb/source/Commands/CommandObjectVersion.cpp +++ b/lldb/source/Commands/CommandObjectVersion.cpp @@ -20,7 +20,7 @@ : CommandObjectParsed(interpreter, "version", "Show the LLDB debugger version.", "version") {} -CommandObjectVersion::~CommandObjectVersion() {} +CommandObjectVersion::~CommandObjectVersion() = default; bool CommandObjectVersion::DoExecute(Args &args, CommandReturnObject &result) { if (args.GetArgumentCount() == 0) { diff --git a/lldb/source/Core/AddressRange.cpp b/lldb/source/Core/AddressRange.cpp --- a/lldb/source/Core/AddressRange.cpp +++ b/lldb/source/Core/AddressRange.cpp @@ -41,7 +41,7 @@ AddressRange::AddressRange(const Address &so_addr, addr_t byte_size) : m_base_addr(so_addr), m_byte_size(byte_size) {} -AddressRange::~AddressRange() {} +AddressRange::~AddressRange() = default; bool AddressRange::Contains(const Address &addr) const { SectionSP range_sect_sp = GetBaseAddress().GetSection(); diff --git a/lldb/source/Core/AddressResolver.cpp b/lldb/source/Core/AddressResolver.cpp --- a/lldb/source/Core/AddressResolver.cpp +++ b/lldb/source/Core/AddressResolver.cpp @@ -17,9 +17,9 @@ using namespace lldb_private; // AddressResolver: -AddressResolver::AddressResolver() {} +AddressResolver::AddressResolver() = default; -AddressResolver::~AddressResolver() {} +AddressResolver::~AddressResolver() = default; void AddressResolver::ResolveAddressInModules(SearchFilter &filter, ModuleList &modules) { diff --git a/lldb/source/Core/AddressResolverFileLine.cpp b/lldb/source/Core/AddressResolverFileLine.cpp --- a/lldb/source/Core/AddressResolverFileLine.cpp +++ b/lldb/source/Core/AddressResolverFileLine.cpp @@ -32,7 +32,7 @@ SourceLocationSpec location_spec) : AddressResolver(), m_src_location_spec(location_spec) {} -AddressResolverFileLine::~AddressResolverFileLine() {} +AddressResolverFileLine::~AddressResolverFileLine() = default; Searcher::CallbackReturn AddressResolverFileLine::SearchCallback(SearchFilter &filter, diff --git a/lldb/source/Core/FileLineResolver.cpp b/lldb/source/Core/FileLineResolver.cpp --- a/lldb/source/Core/FileLineResolver.cpp +++ b/lldb/source/Core/FileLineResolver.cpp @@ -29,7 +29,7 @@ : Searcher(), m_file_spec(file_spec), m_line_number(line_no), m_inlines(check_inlines) {} -FileLineResolver::~FileLineResolver() {} +FileLineResolver::~FileLineResolver() = default; Searcher::CallbackReturn FileLineResolver::SearchCallback(SearchFilter &filter, SymbolContext &context, diff --git a/lldb/source/Core/ModuleChild.cpp b/lldb/source/Core/ModuleChild.cpp --- a/lldb/source/Core/ModuleChild.cpp +++ b/lldb/source/Core/ModuleChild.cpp @@ -13,7 +13,7 @@ ModuleChild::ModuleChild(const lldb::ModuleSP &module_sp) : m_module_wp(module_sp) {} -ModuleChild::~ModuleChild() {} +ModuleChild::~ModuleChild() = default; const ModuleChild &ModuleChild::operator=(const ModuleChild &rhs) { if (this != &rhs) diff --git a/lldb/source/Core/PluginManager.cpp b/lldb/source/Core/PluginManager.cpp --- a/lldb/source/Core/PluginManager.cpp +++ b/lldb/source/Core/PluginManager.cpp @@ -38,7 +38,7 @@ typedef void (*PluginTermCallback)(); struct PluginInfo { - PluginInfo() {} + PluginInfo() = default; llvm::sys::DynamicLibrary library; PluginInitCallback plugin_init_callback = nullptr; diff --git a/lldb/source/Core/SourceManager.cpp b/lldb/source/Core/SourceManager.cpp --- a/lldb/source/Core/SourceManager.cpp +++ b/lldb/source/Core/SourceManager.cpp @@ -61,7 +61,7 @@ m_debugger_wp(debugger_sp) {} // Destructor -SourceManager::~SourceManager() {} +SourceManager::~SourceManager() = default; SourceManager::FileSP SourceManager::GetFile(const FileSpec &file_spec) { if (!file_spec) diff --git a/lldb/source/Core/StreamFile.cpp b/lldb/source/Core/StreamFile.cpp --- a/lldb/source/Core/StreamFile.cpp +++ b/lldb/source/Core/StreamFile.cpp @@ -43,7 +43,7 @@ } } -StreamFile::~StreamFile() {} +StreamFile::~StreamFile() = default; void StreamFile::Flush() { m_file_sp->Flush(); } diff --git a/lldb/source/Core/ValueObject.cpp b/lldb/source/Core/ValueObject.cpp --- a/lldb/source/Core/ValueObject.cpp +++ b/lldb/source/Core/ValueObject.cpp @@ -106,7 +106,7 @@ } // Destructor -ValueObject::~ValueObject() {} +ValueObject::~ValueObject() = default; bool ValueObject::UpdateValueIfNeeded(bool update_format) { @@ -2839,7 +2839,7 @@ const ValueObject::EvaluationPoint &rhs) : m_mod_id(), m_exe_ctx_ref(rhs.m_exe_ctx_ref), m_needs_update(true) {} -ValueObject::EvaluationPoint::~EvaluationPoint() {} +ValueObject::EvaluationPoint::~EvaluationPoint() = default; // This function checks the EvaluationPoint against the current process state. // If the current state matches the evaluation point, or the evaluation point diff --git a/lldb/source/Core/ValueObjectCast.cpp b/lldb/source/Core/ValueObjectCast.cpp --- a/lldb/source/Core/ValueObjectCast.cpp +++ b/lldb/source/Core/ValueObjectCast.cpp @@ -36,7 +36,7 @@ m_value.SetCompilerType(cast_type); } -ValueObjectCast::~ValueObjectCast() {} +ValueObjectCast::~ValueObjectCast() = default; CompilerType ValueObjectCast::GetCompilerTypeImpl() { return m_cast_type; } diff --git a/lldb/source/Core/ValueObjectChild.cpp b/lldb/source/Core/ValueObjectChild.cpp --- a/lldb/source/Core/ValueObjectChild.cpp +++ b/lldb/source/Core/ValueObjectChild.cpp @@ -43,7 +43,7 @@ SetLanguageFlags(language_flags); } -ValueObjectChild::~ValueObjectChild() {} +ValueObjectChild::~ValueObjectChild() = default; lldb::ValueType ValueObjectChild::GetValueType() const { return m_parent->GetValueType(); diff --git a/lldb/source/Core/ValueObjectConstResult.cpp b/lldb/source/Core/ValueObjectConstResult.cpp --- a/lldb/source/Core/ValueObjectConstResult.cpp +++ b/lldb/source/Core/ValueObjectConstResult.cpp @@ -193,7 +193,7 @@ m_error = m_value.GetValueAsData(&exe_ctx, m_data, module); } -ValueObjectConstResult::~ValueObjectConstResult() {} +ValueObjectConstResult::~ValueObjectConstResult() = default; CompilerType ValueObjectConstResult::GetCompilerTypeImpl() { return m_value.GetCompilerType(); diff --git a/lldb/source/Core/ValueObjectConstResultCast.cpp b/lldb/source/Core/ValueObjectConstResultCast.cpp --- a/lldb/source/Core/ValueObjectConstResultCast.cpp +++ b/lldb/source/Core/ValueObjectConstResultCast.cpp @@ -27,7 +27,7 @@ m_name = name; } -ValueObjectConstResultCast::~ValueObjectConstResultCast() {} +ValueObjectConstResultCast::~ValueObjectConstResultCast() = default; lldb::ValueObjectSP ValueObjectConstResultCast::Dereference(Status &error) { return m_impl.Dereference(error); diff --git a/lldb/source/Core/ValueObjectConstResultChild.cpp b/lldb/source/Core/ValueObjectConstResultChild.cpp --- a/lldb/source/Core/ValueObjectConstResultChild.cpp +++ b/lldb/source/Core/ValueObjectConstResultChild.cpp @@ -34,7 +34,7 @@ m_name = name; } -ValueObjectConstResultChild::~ValueObjectConstResultChild() {} +ValueObjectConstResultChild::~ValueObjectConstResultChild() = default; lldb::ValueObjectSP ValueObjectConstResultChild::Dereference(Status &error) { return m_impl.Dereference(error); diff --git a/lldb/source/Core/ValueObjectMemory.cpp b/lldb/source/Core/ValueObjectMemory.cpp --- a/lldb/source/Core/ValueObjectMemory.cpp +++ b/lldb/source/Core/ValueObjectMemory.cpp @@ -106,7 +106,7 @@ } } -ValueObjectMemory::~ValueObjectMemory() {} +ValueObjectMemory::~ValueObjectMemory() = default; CompilerType ValueObjectMemory::GetCompilerTypeImpl() { if (m_type_sp) diff --git a/lldb/source/Core/ValueObjectRegister.cpp b/lldb/source/Core/ValueObjectRegister.cpp --- a/lldb/source/Core/ValueObjectRegister.cpp +++ b/lldb/source/Core/ValueObjectRegister.cpp @@ -60,7 +60,7 @@ } } -ValueObjectRegisterSet::~ValueObjectRegisterSet() {} +ValueObjectRegisterSet::~ValueObjectRegisterSet() = default; CompilerType ValueObjectRegisterSet::GetCompilerTypeImpl() { return CompilerType(); @@ -193,7 +193,7 @@ ConstructObject(reg_num); } -ValueObjectRegister::~ValueObjectRegister() {} +ValueObjectRegister::~ValueObjectRegister() = default; CompilerType ValueObjectRegister::GetCompilerTypeImpl() { if (!m_compiler_type.IsValid()) { diff --git a/lldb/source/Core/ValueObjectVariable.cpp b/lldb/source/Core/ValueObjectVariable.cpp --- a/lldb/source/Core/ValueObjectVariable.cpp +++ b/lldb/source/Core/ValueObjectVariable.cpp @@ -63,7 +63,7 @@ m_name = var_sp->GetName(); } -ValueObjectVariable::~ValueObjectVariable() {} +ValueObjectVariable::~ValueObjectVariable() = default; CompilerType ValueObjectVariable::GetCompilerTypeImpl() { Type *var_type = m_variable_sp->GetType(); diff --git a/lldb/source/DataFormatters/TypeFormat.cpp b/lldb/source/DataFormatters/TypeFormat.cpp --- a/lldb/source/DataFormatters/TypeFormat.cpp +++ b/lldb/source/DataFormatters/TypeFormat.cpp @@ -29,13 +29,13 @@ TypeFormatImpl::TypeFormatImpl(const Flags &flags) : m_flags(flags) {} -TypeFormatImpl::~TypeFormatImpl() {} +TypeFormatImpl::~TypeFormatImpl() = default; TypeFormatImpl_Format::TypeFormatImpl_Format(lldb::Format f, const TypeFormatImpl::Flags &flags) : TypeFormatImpl(flags), m_format(f) {} -TypeFormatImpl_Format::~TypeFormatImpl_Format() {} +TypeFormatImpl_Format::~TypeFormatImpl_Format() = default; bool TypeFormatImpl_Format::FormatObject(ValueObject *valobj, std::string &dest) const { @@ -134,7 +134,7 @@ ConstString type_name, const TypeFormatImpl::Flags &flags) : TypeFormatImpl(flags), m_enum_type(type_name), m_types() {} -TypeFormatImpl_EnumType::~TypeFormatImpl_EnumType() {} +TypeFormatImpl_EnumType::~TypeFormatImpl_EnumType() = default; bool TypeFormatImpl_EnumType::FormatObject(ValueObject *valobj, std::string &dest) const { diff --git a/lldb/source/DataFormatters/TypeSummary.cpp b/lldb/source/DataFormatters/TypeSummary.cpp --- a/lldb/source/DataFormatters/TypeSummary.cpp +++ b/lldb/source/DataFormatters/TypeSummary.cpp @@ -26,7 +26,7 @@ using namespace lldb; using namespace lldb_private; -TypeSummaryOptions::TypeSummaryOptions() {} +TypeSummaryOptions::TypeSummaryOptions() = default; lldb::LanguageType TypeSummaryOptions::GetLanguage() const { return m_lang; } diff --git a/lldb/source/DataFormatters/TypeSynthetic.cpp b/lldb/source/DataFormatters/TypeSynthetic.cpp --- a/lldb/source/DataFormatters/TypeSynthetic.cpp +++ b/lldb/source/DataFormatters/TypeSynthetic.cpp @@ -143,7 +143,7 @@ m_python_class.c_str(), backend.GetSP()); } -ScriptedSyntheticChildren::FrontEnd::~FrontEnd() {} +ScriptedSyntheticChildren::FrontEnd::~FrontEnd() = default; lldb::ValueObjectSP ScriptedSyntheticChildren::FrontEnd::GetChildAtIndex(size_t idx) { diff --git a/lldb/source/Expression/DWARFExpression.cpp b/lldb/source/Expression/DWARFExpression.cpp --- a/lldb/source/Expression/DWARFExpression.cpp +++ b/lldb/source/Expression/DWARFExpression.cpp @@ -67,7 +67,7 @@ } // Destructor -DWARFExpression::~DWARFExpression() {} +DWARFExpression::~DWARFExpression() = default; bool DWARFExpression::IsValid() const { return m_data.GetByteSize() > 0; } diff --git a/lldb/source/Expression/ExpressionVariable.cpp b/lldb/source/Expression/ExpressionVariable.cpp --- a/lldb/source/Expression/ExpressionVariable.cpp +++ b/lldb/source/Expression/ExpressionVariable.cpp @@ -13,7 +13,7 @@ using namespace lldb_private; -ExpressionVariable::~ExpressionVariable() {} +ExpressionVariable::~ExpressionVariable() = default; uint8_t *ExpressionVariable::GetValueBytes() { llvm::Optional byte_size = m_frozen_sp->GetByteSize(); @@ -28,7 +28,7 @@ return nullptr; } -PersistentExpressionState::~PersistentExpressionState() {} +PersistentExpressionState::~PersistentExpressionState() = default; lldb::addr_t PersistentExpressionState::LookupSymbol(ConstString name) { SymbolMap::iterator si = m_symbol_map.find(name.GetCString()); diff --git a/lldb/source/Expression/IRExecutionUnit.cpp b/lldb/source/Expression/IRExecutionUnit.cpp --- a/lldb/source/Expression/IRExecutionUnit.cpp +++ b/lldb/source/Expression/IRExecutionUnit.cpp @@ -496,7 +496,7 @@ IRExecutionUnit::MemoryManager::MemoryManager(IRExecutionUnit &parent) : m_default_mm_up(new llvm::SectionMemoryManager()), m_parent(parent) {} -IRExecutionUnit::MemoryManager::~MemoryManager() {} +IRExecutionUnit::MemoryManager::~MemoryManager() = default; lldb::SectionType IRExecutionUnit::GetSectionTypeFromSectionName( const llvm::StringRef &name, IRExecutionUnit::AllocationKind alloc_kind) { diff --git a/lldb/source/Expression/IRInterpreter.cpp b/lldb/source/Expression/IRInterpreter.cpp --- a/lldb/source/Expression/IRInterpreter.cpp +++ b/lldb/source/Expression/IRInterpreter.cpp @@ -122,7 +122,7 @@ m_stack_pointer = stack_frame_top; } - ~InterpreterStackFrame() {} + ~InterpreterStackFrame() = default; void Jump(const BasicBlock *bb) { m_prev_bb = m_bb; diff --git a/lldb/source/Expression/UserExpression.cpp b/lldb/source/Expression/UserExpression.cpp --- a/lldb/source/Expression/UserExpression.cpp +++ b/lldb/source/Expression/UserExpression.cpp @@ -57,7 +57,7 @@ m_expr_prefix(std::string(prefix)), m_language(language), m_desired_type(desired_type), m_options(options) {} -UserExpression::~UserExpression() {} +UserExpression::~UserExpression() = default; void UserExpression::InstallContext(ExecutionContext &exe_ctx) { m_jit_process_wp = exe_ctx.GetProcessSP(); diff --git a/lldb/source/Host/common/HostNativeThreadBase.cpp b/lldb/source/Host/common/HostNativeThreadBase.cpp --- a/lldb/source/Host/common/HostNativeThreadBase.cpp +++ b/lldb/source/Host/common/HostNativeThreadBase.cpp @@ -17,9 +17,6 @@ using namespace lldb; using namespace lldb_private; -HostNativeThreadBase::HostNativeThreadBase() - : m_thread(LLDB_INVALID_HOST_THREAD) {} - HostNativeThreadBase::HostNativeThreadBase(thread_t thread) : m_thread(thread), m_result(0) {} diff --git a/lldb/source/Host/common/HostProcess.cpp b/lldb/source/Host/common/HostProcess.cpp --- a/lldb/source/Host/common/HostProcess.cpp +++ b/lldb/source/Host/common/HostProcess.cpp @@ -18,7 +18,7 @@ HostProcess::HostProcess(lldb::process_t process) : m_native_process(new HostNativeProcess(process)) {} -HostProcess::~HostProcess() {} +HostProcess::~HostProcess() = default; Status HostProcess::Terminate() { return m_native_process->Terminate(); } diff --git a/lldb/source/Host/common/NativeRegisterContext.cpp b/lldb/source/Host/common/NativeRegisterContext.cpp --- a/lldb/source/Host/common/NativeRegisterContext.cpp +++ b/lldb/source/Host/common/NativeRegisterContext.cpp @@ -22,7 +22,7 @@ : m_thread(thread) {} // Destructor -NativeRegisterContext::~NativeRegisterContext() {} +NativeRegisterContext::~NativeRegisterContext() = default; // FIXME revisit invalidation, process stop ids, etc. Right now we don't // support caching in NativeRegisterContext. We can do this later by utilizing diff --git a/lldb/source/Host/common/PseudoTerminal.cpp b/lldb/source/Host/common/PseudoTerminal.cpp --- a/lldb/source/Host/common/PseudoTerminal.cpp +++ b/lldb/source/Host/common/PseudoTerminal.cpp @@ -29,7 +29,7 @@ using namespace lldb_private; // PseudoTerminal constructor -PseudoTerminal::PseudoTerminal() {} +PseudoTerminal::PseudoTerminal() = default; // Destructor // diff --git a/lldb/source/Host/common/SocketAddress.cpp b/lldb/source/Host/common/SocketAddress.cpp --- a/lldb/source/Host/common/SocketAddress.cpp +++ b/lldb/source/Host/common/SocketAddress.cpp @@ -93,7 +93,7 @@ } // Destructor -SocketAddress::~SocketAddress() {} +SocketAddress::~SocketAddress() = default; void SocketAddress::Clear() { memset(&m_socket_addr, 0, sizeof(m_socket_addr)); diff --git a/lldb/source/Host/common/Terminal.cpp b/lldb/source/Host/common/Terminal.cpp --- a/lldb/source/Host/common/Terminal.cpp +++ b/lldb/source/Host/common/Terminal.cpp @@ -92,7 +92,7 @@ } // Destructor -TerminalState::~TerminalState() {} +TerminalState::~TerminalState() = default; void TerminalState::Clear() { m_tty.Clear(); @@ -190,10 +190,10 @@ } // Constructor -TerminalStateSwitcher::TerminalStateSwitcher() {} +TerminalStateSwitcher::TerminalStateSwitcher() = default; // Destructor -TerminalStateSwitcher::~TerminalStateSwitcher() {} +TerminalStateSwitcher::~TerminalStateSwitcher() = default; // Returns the number of states that this switcher contains uint32_t TerminalStateSwitcher::GetNumberOfStates() const { diff --git a/lldb/source/Host/common/XML.cpp b/lldb/source/Host/common/XML.cpp --- a/lldb/source/Host/common/XML.cpp +++ b/lldb/source/Host/common/XML.cpp @@ -17,7 +17,7 @@ #pragma mark-- XMLDocument -XMLDocument::XMLDocument() {} +XMLDocument::XMLDocument() = default; XMLDocument::~XMLDocument() { Clear(); } @@ -91,11 +91,11 @@ #pragma mark-- XMLNode -XMLNode::XMLNode() {} +XMLNode::XMLNode() = default; XMLNode::XMLNode(XMLNodeImpl node) : m_node(node) {} -XMLNode::~XMLNode() {} +XMLNode::~XMLNode() = default; void XMLNode::Clear() { m_node = nullptr; } @@ -398,7 +398,7 @@ ParseFile(path); } -ApplePropertyList::~ApplePropertyList() {} +ApplePropertyList::~ApplePropertyList() = default; llvm::StringRef ApplePropertyList::GetErrors() const { return m_xml_doc.GetErrors(); diff --git a/lldb/source/Host/macosx/cfcpp/CFCBundle.cpp b/lldb/source/Host/macosx/cfcpp/CFCBundle.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCBundle.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCBundle.cpp @@ -19,7 +19,7 @@ : CFCReleaser(url ? CFBundleCreate(NULL, url) : NULL) {} // Destructor -CFCBundle::~CFCBundle() {} +CFCBundle::~CFCBundle() = default; // Set the path for a bundle by supplying a bool CFCBundle::SetPath(const char *path) { diff --git a/lldb/source/Host/macosx/cfcpp/CFCData.cpp b/lldb/source/Host/macosx/cfcpp/CFCData.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCData.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCData.cpp @@ -24,7 +24,7 @@ } // Destructor -CFCData::~CFCData() {} +CFCData::~CFCData() = default; CFIndex CFCData::GetLength() const { CFDataRef data = get(); diff --git a/lldb/source/Host/macosx/cfcpp/CFCMutableArray.cpp b/lldb/source/Host/macosx/cfcpp/CFCMutableArray.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCMutableArray.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCMutableArray.cpp @@ -29,7 +29,7 @@ } // Destructor -CFCMutableArray::~CFCMutableArray() {} +CFCMutableArray::~CFCMutableArray() = default; CFIndex CFCMutableArray::GetCount() const { CFMutableArrayRef array = get(); diff --git a/lldb/source/Host/macosx/cfcpp/CFCMutableDictionary.cpp b/lldb/source/Host/macosx/cfcpp/CFCMutableDictionary.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCMutableDictionary.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCMutableDictionary.cpp @@ -25,7 +25,7 @@ } // Destructor -CFCMutableDictionary::~CFCMutableDictionary() {} +CFCMutableDictionary::~CFCMutableDictionary() = default; CFIndex CFCMutableDictionary::GetCount() const { CFMutableDictionaryRef dict = get(); diff --git a/lldb/source/Host/macosx/cfcpp/CFCMutableSet.cpp b/lldb/source/Host/macosx/cfcpp/CFCMutableSet.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCMutableSet.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCMutableSet.cpp @@ -25,7 +25,7 @@ } // Destructor -CFCMutableSet::~CFCMutableSet() {} +CFCMutableSet::~CFCMutableSet() = default; CFIndex CFCMutableSet::GetCount() const { CFMutableSetRef set = get(); diff --git a/lldb/source/Host/macosx/cfcpp/CFCString.cpp b/lldb/source/Host/macosx/cfcpp/CFCString.cpp --- a/lldb/source/Host/macosx/cfcpp/CFCString.cpp +++ b/lldb/source/Host/macosx/cfcpp/CFCString.cpp @@ -32,7 +32,7 @@ } // Destructor -CFCString::~CFCString() {} +CFCString::~CFCString() = default; const char *CFCString::GetFileSystemRepresentation(std::string &s) { return CFCString::FileSystemRepresentation(get(), s); diff --git a/lldb/source/Host/posix/HostProcessPosix.cpp b/lldb/source/Host/posix/HostProcessPosix.cpp --- a/lldb/source/Host/posix/HostProcessPosix.cpp +++ b/lldb/source/Host/posix/HostProcessPosix.cpp @@ -28,7 +28,7 @@ HostProcessPosix::HostProcessPosix(lldb::process_t process) : HostNativeProcessBase(process) {} -HostProcessPosix::~HostProcessPosix() {} +HostProcessPosix::~HostProcessPosix() = default; Status HostProcessPosix::Signal(int signo) const { if (m_process == kInvalidPosixProcess) { diff --git a/lldb/source/Host/posix/HostThreadPosix.cpp b/lldb/source/Host/posix/HostThreadPosix.cpp --- a/lldb/source/Host/posix/HostThreadPosix.cpp +++ b/lldb/source/Host/posix/HostThreadPosix.cpp @@ -15,12 +15,12 @@ using namespace lldb; using namespace lldb_private; -HostThreadPosix::HostThreadPosix() {} +HostThreadPosix::HostThreadPosix() = default; HostThreadPosix::HostThreadPosix(lldb::thread_t thread) : HostNativeThreadBase(thread) {} -HostThreadPosix::~HostThreadPosix() {} +HostThreadPosix::~HostThreadPosix() = default; Status HostThreadPosix::Join(lldb::thread_result_t *result) { Status error; diff --git a/lldb/source/Initialization/SystemInitializer.cpp b/lldb/source/Initialization/SystemInitializer.cpp --- a/lldb/source/Initialization/SystemInitializer.cpp +++ b/lldb/source/Initialization/SystemInitializer.cpp @@ -10,6 +10,6 @@ using namespace lldb_private; -SystemInitializer::SystemInitializer() {} +SystemInitializer::SystemInitializer() = default; -SystemInitializer::~SystemInitializer() {} +SystemInitializer::~SystemInitializer() = default; diff --git a/lldb/source/Initialization/SystemInitializerCommon.cpp b/lldb/source/Initialization/SystemInitializerCommon.cpp --- a/lldb/source/Initialization/SystemInitializerCommon.cpp +++ b/lldb/source/Initialization/SystemInitializerCommon.cpp @@ -38,7 +38,7 @@ HostInfo::SharedLibraryDirectoryHelper *helper) : m_shlib_dir_helper(helper) {} -SystemInitializerCommon::~SystemInitializerCommon() {} +SystemInitializerCommon::~SystemInitializerCommon() = default; /// Initialize the FileSystem based on the current reproducer mode. static llvm::Error InitializeFileSystem() { diff --git a/lldb/source/Interpreter/OptionGroupFile.cpp b/lldb/source/Interpreter/OptionGroupFile.cpp --- a/lldb/source/Interpreter/OptionGroupFile.cpp +++ b/lldb/source/Interpreter/OptionGroupFile.cpp @@ -59,7 +59,7 @@ m_option_definition.usage_text = usage_text; } -OptionGroupFileList::~OptionGroupFileList() {} +OptionGroupFileList::~OptionGroupFileList() = default; Status OptionGroupFileList::SetOptionValue(uint32_t option_idx, diff --git a/lldb/source/Interpreter/OptionValueFileColonLine.cpp b/lldb/source/Interpreter/OptionValueFileColonLine.cpp --- a/lldb/source/Interpreter/OptionValueFileColonLine.cpp +++ b/lldb/source/Interpreter/OptionValueFileColonLine.cpp @@ -21,9 +21,7 @@ // I set the completer to "source file" which isn't quite right, but we can // only usefully complete in the file name part of it so it should be good // enough. -OptionValueFileColonLine::OptionValueFileColonLine() - -{} +OptionValueFileColonLine::OptionValueFileColonLine() = default; OptionValueFileColonLine::OptionValueFileColonLine(llvm::StringRef input) : m_line_number(LLDB_INVALID_LINE_NUMBER), @@ -91,8 +89,8 @@ llvm::StringRef middle_piece; std::tie(file_name, middle_piece) = left_of_last_piece.rsplit(':'); - if (middle_piece.empty() || !llvm::to_integer(middle_piece, - m_line_number)) { + if (middle_piece.empty() || + !llvm::to_integer(middle_piece, m_line_number)) { // The middle piece was empty or not an integer, so there were only two // legit pieces; our original division was right. Reassign the file // name and pull out the line number: diff --git a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.h b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.h --- a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.h +++ b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.h @@ -214,9 +214,7 @@ uint32_t entry_count = 0; lldb::addr_t image_infos_addr = LLDB_INVALID_ADDRESS; - OSKextLoadedKextSummaryHeader() - - {} + OSKextLoadedKextSummaryHeader() = default; uint32_t GetSize() { switch (version) { diff --git a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp --- a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp +++ b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp @@ -107,7 +107,7 @@ m_collection_sp->Initialize(g_dynamicloaderdarwinkernel_properties); } - ~DynamicLoaderDarwinKernelProperties() override {} + ~DynamicLoaderDarwinKernelProperties() override = default; bool GetLoadKexts() const { const uint32_t idx = ePropertyLoadKexts; diff --git a/lldb/source/Plugins/DynamicLoader/Hexagon-DYLD/HexagonDYLDRendezvous.h b/lldb/source/Plugins/DynamicLoader/Hexagon-DYLD/HexagonDYLDRendezvous.h --- a/lldb/source/Plugins/DynamicLoader/Hexagon-DYLD/HexagonDYLDRendezvous.h +++ b/lldb/source/Plugins/DynamicLoader/Hexagon-DYLD/HexagonDYLDRendezvous.h @@ -41,7 +41,7 @@ uint64_t state = 0; lldb::addr_t ldbase = 0; - Rendezvous() {} + Rendezvous() = default; }; public: diff --git a/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp b/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp --- a/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp +++ b/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp @@ -60,7 +60,7 @@ m_dyld_image_infos_stop_id(UINT32_MAX), m_dyld(), m_mutex() {} // Destructor -DynamicLoaderDarwin::~DynamicLoaderDarwin() {} +DynamicLoaderDarwin::~DynamicLoaderDarwin() = default; /// Called after attaching a process. /// diff --git a/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.h b/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.h --- a/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.h +++ b/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.h @@ -106,9 +106,7 @@ bool libSystemInitialized = false; // Version >= 2 lldb::addr_t dyldImageLoadAddress = LLDB_INVALID_ADDRESS; // Version >= 2 - DYLDAllImageInfos() - - {} + DYLDAllImageInfos() = default; void Clear() { version = 0; diff --git a/lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.h b/lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.h --- a/lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.h +++ b/lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DYLDRendezvous.h @@ -44,7 +44,7 @@ uint64_t state = 0; lldb::addr_t ldbase = 0; - Rendezvous() {} + Rendezvous() = default; }; public: diff --git a/lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp b/lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp --- a/lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp +++ b/lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp @@ -28,7 +28,7 @@ DynamicLoaderWindowsDYLD::DynamicLoaderWindowsDYLD(Process *process) : DynamicLoader(process) {} -DynamicLoaderWindowsDYLD::~DynamicLoaderWindowsDYLD() {} +DynamicLoaderWindowsDYLD::~DynamicLoaderWindowsDYLD() = default; void DynamicLoaderWindowsDYLD::Initialize() { PluginManager::RegisterPlugin(GetPluginNameStatic(), diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ASTResultSynthesizer.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ASTResultSynthesizer.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ASTResultSynthesizer.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ASTResultSynthesizer.cpp @@ -43,7 +43,7 @@ m_passthrough_sema = dyn_cast(passthrough); } -ASTResultSynthesizer::~ASTResultSynthesizer() {} +ASTResultSynthesizer::~ASTResultSynthesizer() = default; void ASTResultSynthesizer::Initialize(ASTContext &Context) { m_ast_context = &Context; diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ASTStructExtractor.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ASTStructExtractor.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ASTStructExtractor.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ASTStructExtractor.cpp @@ -38,7 +38,7 @@ m_passthrough_sema = dyn_cast(passthrough); } -ASTStructExtractor::~ASTStructExtractor() {} +ASTStructExtractor::~ASTStructExtractor() = default; void ASTStructExtractor::Initialize(ASTContext &Context) { m_ast_context = &Context; diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ASTUtils.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ASTUtils.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ASTUtils.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ASTUtils.cpp @@ -8,17 +8,17 @@ #include "ASTUtils.h" -lldb_private::ExternalASTSourceWrapper::~ExternalASTSourceWrapper() {} +lldb_private::ExternalASTSourceWrapper::~ExternalASTSourceWrapper() = default; void lldb_private::ExternalASTSourceWrapper::PrintStats() { m_Source->PrintStats(); } -lldb_private::ASTConsumerForwarder::~ASTConsumerForwarder() {} +lldb_private::ASTConsumerForwarder::~ASTConsumerForwarder() = default; void lldb_private::ASTConsumerForwarder::PrintStats() { m_c->PrintStats(); } -lldb_private::SemaSourceWithPriorities::~SemaSourceWithPriorities() {} +lldb_private::SemaSourceWithPriorities::~SemaSourceWithPriorities() = default; void lldb_private::SemaSourceWithPriorities::PrintStats() { for (size_t i = 0; i < Sources.size(); ++i) diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.h @@ -216,7 +216,7 @@ void ForgetSource(clang::ASTContext *dst_ctx, clang::ASTContext *src_ctx); struct DeclOrigin { - DeclOrigin() {} + DeclOrigin() = default; DeclOrigin(clang::ASTContext *_ctx, clang::Decl *_decl) : ctx(_ctx), decl(_decl) { diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTImporter.cpp @@ -184,7 +184,7 @@ } public: - DeclContextOverride() {} + DeclContextOverride() = default; void OverrideAllDeclsFromContainingFunction(clang::Decl *decl) { for (DeclContext *decl_context = decl->getLexicalDeclContext(); diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangDeclVendor.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangDeclVendor.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangDeclVendor.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangDeclVendor.h @@ -22,7 +22,7 @@ public: ClangDeclVendor(DeclVendorKind kind) : DeclVendor(kind) {} - virtual ~ClangDeclVendor() {} + virtual ~ClangDeclVendor() = default; using DeclVendor::FindDecls; diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.h @@ -251,7 +251,7 @@ lldb::ByteOrder byte_order = lldb::eByteOrderInvalid; size_t address_byte_size = 0; - TargetInfo() {} + TargetInfo() = default; bool IsValid() { return (byte_order != lldb::eByteOrderInvalid && address_byte_size != 0); @@ -308,7 +308,7 @@ /// The following values should not live beyond parsing class ParserVars { public: - ParserVars() {} + ParserVars() = default; Target *GetTarget() { if (m_exe_ctx.GetTargetPtr()) diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionHelper.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionHelper.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionHelper.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionHelper.h @@ -38,7 +38,7 @@ ExpressionTypeSystemHelper::LLVMCastKind::eKindClangHelper) {} /// Destructor - virtual ~ClangExpressionHelper() {} + virtual ~ClangExpressionHelper() = default; /// Return the object that the parser should use when resolving external /// values. May be NULL if everything should be self-contained. diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp @@ -725,7 +725,7 @@ m_compiler->getCodeGenOpts(), *m_llvm_context)); } -ClangExpressionParser::~ClangExpressionParser() {} +ClangExpressionParser::~ClangExpressionParser() = default; namespace { diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionVariable.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionVariable.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionVariable.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionVariable.h @@ -157,7 +157,7 @@ /// The following values are valid if the variable is used by JIT code struct JITVars { - JITVars() {} + JITVars() = default; lldb::offset_t m_alignment = 0; ///< The required alignment of the variable, in bytes diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangFunctionCaller.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangFunctionCaller.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangFunctionCaller.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangFunctionCaller.cpp @@ -59,7 +59,7 @@ } // Destructor -ClangFunctionCaller::~ClangFunctionCaller() {} +ClangFunctionCaller::~ClangFunctionCaller() = default; unsigned diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp @@ -176,7 +176,7 @@ ClangModulesDeclVendor::ClangModulesDeclVendor() : ClangDeclVendor(eClangModuleDeclVendor) {} -ClangModulesDeclVendor::~ClangModulesDeclVendor() {} +ClangModulesDeclVendor::~ClangModulesDeclVendor() = default; ClangModulesDeclVendorImpl::ClangModulesDeclVendorImpl( llvm::IntrusiveRefCntPtr diagnostics_engine, diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp @@ -90,7 +90,7 @@ } } -ClangUserExpression::~ClangUserExpression() {} +ClangUserExpression::~ClangUserExpression() = default; void ClangUserExpression::ScanContext(ExecutionContext &exe_ctx, Status &err) { Log *log(lldb_private::GetLogIfAllCategoriesSet(LIBLLDB_LOG_EXPRESSIONS)); diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.h --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.h @@ -74,9 +74,9 @@ private: class ClangUtilityFunctionHelper : public ClangExpressionHelper { public: - ClangUtilityFunctionHelper() {} + ClangUtilityFunctionHelper() = default; - ~ClangUtilityFunctionHelper() override {} + ~ClangUtilityFunctionHelper() override = default; /// Return the object that the parser should use when resolving external /// values. May be NULL if everything should be self-contained. diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangUtilityFunction.cpp @@ -65,7 +65,7 @@ } } -ClangUtilityFunction::~ClangUtilityFunction() {} +ClangUtilityFunction::~ClangUtilityFunction() = default; /// Install the utility function into a process /// diff --git a/lldb/source/Plugins/ExpressionParser/Clang/CppModuleConfiguration.h b/lldb/source/Plugins/ExpressionParser/Clang/CppModuleConfiguration.h --- a/lldb/source/Plugins/ExpressionParser/Clang/CppModuleConfiguration.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/CppModuleConfiguration.h @@ -59,7 +59,7 @@ /// Creates a configuration by analyzing the given list of used source files. explicit CppModuleConfiguration(const FileSpecList &support_files); /// Creates an empty and invalid configuration. - CppModuleConfiguration() {} + CppModuleConfiguration() = default; /// Returns true iff this is a valid configuration that can be used to /// load and compile modules. diff --git a/lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.cpp b/lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.cpp --- a/lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.cpp @@ -46,7 +46,7 @@ IRForTarget::FunctionValueCache::FunctionValueCache(Maker const &maker) : m_maker(maker), m_values() {} -IRForTarget::FunctionValueCache::~FunctionValueCache() {} +IRForTarget::FunctionValueCache::~FunctionValueCache() = default; llvm::Value * IRForTarget::FunctionValueCache::GetValue(llvm::Function *function) { diff --git a/lldb/source/Plugins/Instruction/ARM/EmulateInstructionARM.h b/lldb/source/Plugins/Instruction/ARM/EmulateInstructionARM.h --- a/lldb/source/Plugins/Instruction/ARM/EmulateInstructionARM.h +++ b/lldb/source/Plugins/Instruction/ARM/EmulateInstructionARM.h @@ -19,8 +19,8 @@ // ITSession - Keep track of the IT Block progression. class ITSession { public: - ITSession() {} - ~ITSession() {} + ITSession() = default; + ~ITSession() = default; // InitIT - Initializes ITCounter/ITState. bool InitIT(uint32_t bits7_0); diff --git a/lldb/source/Plugins/Instruction/ARM/EmulationStateARM.cpp b/lldb/source/Plugins/Instruction/ARM/EmulationStateARM.cpp --- a/lldb/source/Plugins/Instruction/ARM/EmulationStateARM.cpp +++ b/lldb/source/Plugins/Instruction/ARM/EmulationStateARM.cpp @@ -24,7 +24,7 @@ ClearPseudoRegisters(); } -EmulationStateARM::~EmulationStateARM() {} +EmulationStateARM::~EmulationStateARM() = default; bool EmulationStateARM::LoadPseudoRegistersFromFrame(StackFrame &frame) { RegisterContext *reg_ctx = frame.GetRegisterContext().get(); diff --git a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.h b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.h --- a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.h +++ b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.h @@ -69,7 +69,7 @@ size_t begin_index = 0; size_t end_index = 0; - Range() {} + Range() = default; Range(size_t begin, size_t end) : begin_index(begin), end_index(end) { assert(end >= begin); } diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCClassDescriptorV2.h b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCClassDescriptorV2.h --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCClassDescriptorV2.h +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCClassDescriptorV2.h @@ -84,7 +84,7 @@ lldb::addr_t m_data_ptr = 0; uint8_t m_flags = 0; - objc_class_t() {} + objc_class_t() = default; void Clear() { m_isa = 0; diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntime.cpp @@ -48,7 +48,7 @@ char AppleObjCRuntime::ID = 0; -AppleObjCRuntime::~AppleObjCRuntime() {} +AppleObjCRuntime::~AppleObjCRuntime() = default; AppleObjCRuntime::AppleObjCRuntime(Process *process) : ObjCLanguageRuntime(process), m_read_objc_library(false), diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.h b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.h --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.h +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.h @@ -126,7 +126,7 @@ class HashTableSignature { public: - HashTableSignature() {} + HashTableSignature() = default; bool NeedsUpdate(uint32_t count, uint32_t num_buckets, lldb::addr_t buckets_ptr) { diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp @@ -1383,7 +1383,7 @@ lldb::addr_t m_invalid_key = 0; }; -AppleObjCRuntimeV2::HashTableSignature::HashTableSignature() {} +AppleObjCRuntimeV2::HashTableSignature::HashTableSignature() = default; void AppleObjCRuntimeV2::HashTableSignature::UpdateSignature( const RemoteNXMapTable &hash_table) { diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.h b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.h --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.h +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.h @@ -76,7 +76,7 @@ class VTableRegion { public: - VTableRegion() {} + VTableRegion() = default; VTableRegion(AppleObjCVTables *owner, lldb::addr_t header_addr); diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp @@ -285,7 +285,7 @@ SetUpRegion(); } -AppleObjCTrampolineHandler::~AppleObjCTrampolineHandler() {} +AppleObjCTrampolineHandler::~AppleObjCTrampolineHandler() = default; void AppleObjCTrampolineHandler::AppleObjCVTables::VTableRegion::SetUpRegion() { // The header looks like: diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.cpp b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.cpp --- a/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.cpp +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.cpp @@ -40,7 +40,7 @@ // Destructor AppleThreadPlanStepThroughObjCTrampoline:: - ~AppleThreadPlanStepThroughObjCTrampoline() {} + ~AppleThreadPlanStepThroughObjCTrampoline() = default; void AppleThreadPlanStepThroughObjCTrampoline::DidPush() { // Setting up the memory space for the called function text might require diff --git a/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.cpp b/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.cpp --- a/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.cpp +++ b/lldb/source/Plugins/LanguageRuntime/ObjC/ObjCLanguageRuntime.cpp @@ -32,7 +32,7 @@ char ObjCLanguageRuntime::ID = 0; // Destructor -ObjCLanguageRuntime::~ObjCLanguageRuntime() {} +ObjCLanguageRuntime::~ObjCLanguageRuntime() = default; ObjCLanguageRuntime::ObjCLanguageRuntime(Process *process) : LanguageRuntime(process), m_impl_cache(), @@ -305,7 +305,7 @@ return CompilerType(); } -ObjCLanguageRuntime::EncodingToType::~EncodingToType() {} +ObjCLanguageRuntime::EncodingToType::~EncodingToType() = default; ObjCLanguageRuntime::EncodingToTypeSP ObjCLanguageRuntime::GetEncodingToType() { return nullptr; @@ -363,7 +363,8 @@ m_class_names.insert(class_name); } -ObjCLanguageRuntime::ObjCExceptionPrecondition::ObjCExceptionPrecondition() {} +ObjCLanguageRuntime::ObjCExceptionPrecondition::ObjCExceptionPrecondition() = + default; bool ObjCLanguageRuntime::ObjCExceptionPrecondition::EvaluatePrecondition( StoppointCallbackContext &context) { diff --git a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptExpressionOpts.cpp b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptExpressionOpts.cpp --- a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptExpressionOpts.cpp +++ b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptExpressionOpts.cpp @@ -185,7 +185,7 @@ EarlyPasses->add(new RenderScriptRuntimeModulePass(process)); } -RSIRPasses::~RSIRPasses() {} +RSIRPasses::~RSIRPasses() = default; } // namespace lldb_renderscript } // namespace lldb_private diff --git a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.h b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.h --- a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.h +++ b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.h @@ -46,7 +46,7 @@ struct RSCoordinate { uint32_t x = 0, y = 0, z = 0; - RSCoordinate(){}; + RSCoordinate() = default; bool operator==(const lldb_renderscript::RSCoordinate &rhs) { return x == rhs.x && y == rhs.y && z == rhs.z; diff --git a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp --- a/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp +++ b/lldb/source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp @@ -61,7 +61,7 @@ template class empirical_type { public: // Ctor. Contents is invalid when constructed. - empirical_type() {} + empirical_type() = default; // Return true and copy contents to out if valid, else return false. bool get(type_t &out) const { diff --git a/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp b/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp --- a/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp +++ b/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp @@ -140,7 +140,7 @@ : m_arch(arch), m_modification_time(time), m_file_offset(file_offset), m_objects(), m_data(data) {} -ObjectContainerBSDArchive::Archive::~Archive() {} +ObjectContainerBSDArchive::Archive::~Archive() = default; size_t ObjectContainerBSDArchive::Archive::ParseObjects() { DataExtractor &data = m_data; @@ -373,7 +373,7 @@ m_archive_sp = archive_sp; } -ObjectContainerBSDArchive::~ObjectContainerBSDArchive() {} +ObjectContainerBSDArchive::~ObjectContainerBSDArchive() = default; bool ObjectContainerBSDArchive::ParseHeader() { if (m_archive_sp.get() == nullptr) { diff --git a/lldb/source/Plugins/ObjectContainer/Universal-Mach-O/ObjectContainerUniversalMachO.cpp b/lldb/source/Plugins/ObjectContainer/Universal-Mach-O/ObjectContainerUniversalMachO.cpp --- a/lldb/source/Plugins/ObjectContainer/Universal-Mach-O/ObjectContainerUniversalMachO.cpp +++ b/lldb/source/Plugins/ObjectContainer/Universal-Mach-O/ObjectContainerUniversalMachO.cpp @@ -79,7 +79,7 @@ memset(&m_header, 0, sizeof(m_header)); } -ObjectContainerUniversalMachO::~ObjectContainerUniversalMachO() {} +ObjectContainerUniversalMachO::~ObjectContainerUniversalMachO() = default; bool ObjectContainerUniversalMachO::ParseHeader() { bool success = ParseHeader(m_data, m_header, m_fat_archs); diff --git a/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.h b/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.h --- a/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.h +++ b/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.h @@ -28,7 +28,7 @@ std::string n_name; - ELFNote() {} + ELFNote() = default; /// Parse an ELFNote entry from the given DataExtractor starting at position /// \p offset. diff --git a/lldb/source/Plugins/ObjectFile/JIT/ObjectFileJIT.cpp b/lldb/source/Plugins/ObjectFile/JIT/ObjectFileJIT.cpp --- a/lldb/source/Plugins/ObjectFile/JIT/ObjectFileJIT.cpp +++ b/lldb/source/Plugins/ObjectFile/JIT/ObjectFileJIT.cpp @@ -100,7 +100,7 @@ } } -ObjectFileJIT::~ObjectFileJIT() {} +ObjectFileJIT::~ObjectFileJIT() = default; bool ObjectFileJIT::ParseHeader() { // JIT code is never in a file, nor is it required to have any header diff --git a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp --- a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp +++ b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp @@ -263,7 +263,7 @@ ::memset(&m_coff_header, 0, sizeof(m_coff_header)); } -ObjectFilePECOFF::~ObjectFilePECOFF() {} +ObjectFilePECOFF::~ObjectFilePECOFF() = default; bool ObjectFilePECOFF::ParseHeader() { ModuleSP module_sp(GetModule()); diff --git a/lldb/source/Plugins/OperatingSystem/Python/OperatingSystemPython.cpp b/lldb/source/Plugins/OperatingSystem/Python/OperatingSystemPython.cpp --- a/lldb/source/Plugins/OperatingSystem/Python/OperatingSystemPython.cpp +++ b/lldb/source/Plugins/OperatingSystem/Python/OperatingSystemPython.cpp @@ -115,7 +115,7 @@ } } -OperatingSystemPython::~OperatingSystemPython() {} +OperatingSystemPython::~OperatingSystemPython() = default; DynamicRegisterInfo *OperatingSystemPython::GetDynamicRegisterInfo() { if (m_register_info_up == nullptr) { diff --git a/lldb/source/Plugins/Platform/Android/AdbClient.cpp b/lldb/source/Plugins/Platform/Android/AdbClient.cpp --- a/lldb/source/Plugins/Platform/Android/AdbClient.cpp +++ b/lldb/source/Plugins/Platform/Android/AdbClient.cpp @@ -118,11 +118,11 @@ return error; } -AdbClient::AdbClient() {} +AdbClient::AdbClient() = default; AdbClient::AdbClient(const std::string &device_id) : m_device_id(device_id) {} -AdbClient::~AdbClient() {} +AdbClient::~AdbClient() = default; void AdbClient::SetDeviceID(const std::string &device_id) { m_device_id = device_id; @@ -583,7 +583,7 @@ return error; } -AdbClient::SyncService::~SyncService() {} +AdbClient::SyncService::~SyncService() = default; Status AdbClient::SyncService::SendSyncRequest(const char *request_id, const uint32_t data_len, diff --git a/lldb/source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp b/lldb/source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp --- a/lldb/source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp +++ b/lldb/source/Plugins/Platform/Android/PlatformAndroidRemoteGDBServer.cpp @@ -73,7 +73,7 @@ return error; } -PlatformAndroidRemoteGDBServer::PlatformAndroidRemoteGDBServer() {} +PlatformAndroidRemoteGDBServer::PlatformAndroidRemoteGDBServer() = default; PlatformAndroidRemoteGDBServer::~PlatformAndroidRemoteGDBServer() { for (const auto &it : m_port_forwards) diff --git a/lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp b/lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp --- a/lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp +++ b/lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp @@ -50,7 +50,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformAppleSimulator::~PlatformAppleSimulator() {} +PlatformAppleSimulator::~PlatformAppleSimulator() = default; lldb_private::Status PlatformAppleSimulator::LaunchProcess( lldb_private::ProcessLaunchInfo &launch_info) { diff --git a/lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp b/lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp --- a/lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp +++ b/lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp @@ -54,7 +54,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformDarwin::~PlatformDarwin() {} +PlatformDarwin::~PlatformDarwin() = default; lldb_private::Status PlatformDarwin::PutFile(const lldb_private::FileSpec &source, diff --git a/lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp b/lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp --- a/lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp +++ b/lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp @@ -197,7 +197,7 @@ m_collection_sp->Initialize(g_platformdarwinkernel_properties); } - virtual ~PlatformDarwinKernelProperties() {} + virtual ~PlatformDarwinKernelProperties() = default; FileSpecList GetKextDirectories() const { const uint32_t idx = ePropertyKextDirectories; @@ -250,7 +250,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformDarwinKernel::~PlatformDarwinKernel() {} +PlatformDarwinKernel::~PlatformDarwinKernel() = default; void PlatformDarwinKernel::GetStatus(Stream &strm) { Platform::GetStatus(strm); diff --git a/lldb/source/Plugins/Platform/MacOSX/PlatformRemoteDarwinDevice.cpp b/lldb/source/Plugins/Platform/MacOSX/PlatformRemoteDarwinDevice.cpp --- a/lldb/source/Plugins/Platform/MacOSX/PlatformRemoteDarwinDevice.cpp +++ b/lldb/source/Plugins/Platform/MacOSX/PlatformRemoteDarwinDevice.cpp @@ -45,7 +45,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformRemoteDarwinDevice::~PlatformRemoteDarwinDevice() {} +PlatformRemoteDarwinDevice::~PlatformRemoteDarwinDevice() = default; void PlatformRemoteDarwinDevice::GetStatus(Stream &strm) { Platform::GetStatus(strm); diff --git a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp --- a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp +++ b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp @@ -46,7 +46,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformPOSIX::~PlatformPOSIX() {} +PlatformPOSIX::~PlatformPOSIX() = default; lldb_private::OptionGroupOptions *PlatformPOSIX::GetConnectionOptions( lldb_private::CommandInterpreter &interpreter) { diff --git a/lldb/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp b/lldb/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp --- a/lldb/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp +++ b/lldb/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp @@ -212,7 +212,7 @@ /// /// The destructor is virtual since this class is designed to be /// inherited from by the plug-in instance. -PlatformRemoteGDBServer::~PlatformRemoteGDBServer() {} +PlatformRemoteGDBServer::~PlatformRemoteGDBServer() = default; bool PlatformRemoteGDBServer::GetSupportedArchitectureAtIndex(uint32_t idx, ArchSpec &arch) { diff --git a/lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp b/lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp --- a/lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp +++ b/lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp @@ -73,7 +73,7 @@ m_collection_sp->Initialize(g_processkdp_properties); } - virtual ~PluginProperties() {} + virtual ~PluginProperties() = default; uint64_t GetPacketTimeout() { const uint32_t idx = ePropertyKDPPacketTimeout; @@ -902,7 +902,7 @@ m_option_group.Finalize(); } - ~CommandObjectProcessKDPPacketSend() {} + ~CommandObjectProcessKDPPacketSend() = default; bool DoExecute(Args &command, CommandReturnObject &result) override { const size_t argc = command.GetArgumentCount(); @@ -1003,7 +1003,7 @@ CommandObjectSP(new CommandObjectProcessKDPPacketSend(interpreter))); } - ~CommandObjectProcessKDPPacket() {} + ~CommandObjectProcessKDPPacket() = default; }; class CommandObjectMultiwordProcessKDP : public CommandObjectMultiword { @@ -1017,7 +1017,7 @@ interpreter))); } - ~CommandObjectMultiwordProcessKDP() {} + ~CommandObjectMultiwordProcessKDP() = default; }; CommandObject *ProcessKDP::GetPluginCommandObject() { diff --git a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm.cpp b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm.cpp --- a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm.cpp +++ b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm.cpp @@ -19,7 +19,7 @@ : RegisterContextDarwin_arm(thread, concrete_frame_idx), m_kdp_thread(thread) {} -RegisterContextKDP_arm::~RegisterContextKDP_arm() {} +RegisterContextKDP_arm::~RegisterContextKDP_arm() = default; int RegisterContextKDP_arm::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { ProcessSP process_sp(CalculateProcess()); diff --git a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm64.cpp b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm64.cpp --- a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm64.cpp +++ b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm64.cpp @@ -19,7 +19,7 @@ : RegisterContextDarwin_arm64(thread, concrete_frame_idx), m_kdp_thread(thread) {} -RegisterContextKDP_arm64::~RegisterContextKDP_arm64() {} +RegisterContextKDP_arm64::~RegisterContextKDP_arm64() = default; int RegisterContextKDP_arm64::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { ProcessSP process_sp(CalculateProcess()); diff --git a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_i386.cpp b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_i386.cpp --- a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_i386.cpp +++ b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_i386.cpp @@ -18,7 +18,7 @@ : RegisterContextDarwin_i386(thread, concrete_frame_idx), m_kdp_thread(thread) {} -RegisterContextKDP_i386::~RegisterContextKDP_i386() {} +RegisterContextKDP_i386::~RegisterContextKDP_i386() = default; int RegisterContextKDP_i386::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { ProcessSP process_sp(CalculateProcess()); diff --git a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_x86_64.cpp b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_x86_64.cpp --- a/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_x86_64.cpp +++ b/lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_x86_64.cpp @@ -18,7 +18,7 @@ : RegisterContextDarwin_x86_64(thread, concrete_frame_idx), m_kdp_thread(thread) {} -RegisterContextKDP_x86_64::~RegisterContextKDP_x86_64() {} +RegisterContextKDP_x86_64::~RegisterContextKDP_x86_64() = default; int RegisterContextKDP_x86_64::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { diff --git a/lldb/source/Plugins/Process/Utility/HistoryUnwind.cpp b/lldb/source/Plugins/Process/Utility/HistoryUnwind.cpp --- a/lldb/source/Plugins/Process/Utility/HistoryUnwind.cpp +++ b/lldb/source/Plugins/Process/Utility/HistoryUnwind.cpp @@ -30,7 +30,7 @@ // Destructor -HistoryUnwind::~HistoryUnwind() {} +HistoryUnwind::~HistoryUnwind() = default; void HistoryUnwind::DoClear() { std::lock_guard guard(m_unwind_mutex); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp @@ -922,7 +922,7 @@ } } -RegisterContextDarwin_arm::~RegisterContextDarwin_arm() {} +RegisterContextDarwin_arm::~RegisterContextDarwin_arm() = default; void RegisterContextDarwin_arm::InvalidateAllRegisters() { InvalidateAllRegisterStates(); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp @@ -104,7 +104,7 @@ } } -RegisterContextDarwin_arm64::~RegisterContextDarwin_arm64() {} +RegisterContextDarwin_arm64::~RegisterContextDarwin_arm64() = default; void RegisterContextDarwin_arm64::InvalidateAllRegisters() { InvalidateAllRegisterStates(); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_i386.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_i386.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_i386.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_i386.cpp @@ -405,7 +405,7 @@ } } -RegisterContextDarwin_i386::~RegisterContextDarwin_i386() {} +RegisterContextDarwin_i386::~RegisterContextDarwin_i386() = default; void RegisterContextDarwin_i386::InvalidateAllRegisters() { InvalidateAllRegisterStates(); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_x86_64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_x86_64.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_x86_64.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_x86_64.cpp @@ -467,7 +467,7 @@ } } -RegisterContextDarwin_x86_64::~RegisterContextDarwin_x86_64() {} +RegisterContextDarwin_x86_64::~RegisterContextDarwin_x86_64() = default; void RegisterContextDarwin_x86_64::InvalidateAllRegisters() { InvalidateAllRegisterStates(); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextFreeBSD_powerpc.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextFreeBSD_powerpc.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextFreeBSD_powerpc.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextFreeBSD_powerpc.cpp @@ -178,7 +178,7 @@ const ArchSpec &target_arch) : RegisterInfoInterface(target_arch) {} -RegisterContextFreeBSD_powerpc::~RegisterContextFreeBSD_powerpc() {} +RegisterContextFreeBSD_powerpc::~RegisterContextFreeBSD_powerpc() = default; size_t RegisterContextFreeBSD_powerpc::GetGPRSize() const { // This is an 'abstract' base, so no GPR struct. @@ -197,7 +197,7 @@ const ArchSpec &target_arch) : RegisterContextFreeBSD_powerpc(target_arch) {} -RegisterContextFreeBSD_powerpc32::~RegisterContextFreeBSD_powerpc32() {} +RegisterContextFreeBSD_powerpc32::~RegisterContextFreeBSD_powerpc32() = default; size_t RegisterContextFreeBSD_powerpc32::GetGPRSize() const { return sizeof(GPR32); @@ -217,7 +217,7 @@ const ArchSpec &target_arch) : RegisterContextFreeBSD_powerpc(target_arch) {} -RegisterContextFreeBSD_powerpc64::~RegisterContextFreeBSD_powerpc64() {} +RegisterContextFreeBSD_powerpc64::~RegisterContextFreeBSD_powerpc64() = default; size_t RegisterContextFreeBSD_powerpc64::GetGPRSize() const { return sizeof(GPR64); diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextMach_arm.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextMach_arm.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextMach_arm.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextMach_arm.cpp @@ -21,7 +21,7 @@ uint32_t concrete_frame_idx) : RegisterContextDarwin_arm(thread, concrete_frame_idx) {} -RegisterContextMach_arm::~RegisterContextMach_arm() {} +RegisterContextMach_arm::~RegisterContextMach_arm() = default; int RegisterContextMach_arm::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { mach_msg_type_number_t count = GPRWordCount; diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextMach_i386.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextMach_i386.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextMach_i386.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextMach_i386.cpp @@ -19,7 +19,7 @@ uint32_t concrete_frame_idx) : RegisterContextDarwin_i386(thread, concrete_frame_idx) {} -RegisterContextMach_i386::~RegisterContextMach_i386() {} +RegisterContextMach_i386::~RegisterContextMach_i386() = default; int RegisterContextMach_i386::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { mach_msg_type_number_t count = GPRWordCount; diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextMach_x86_64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextMach_x86_64.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextMach_x86_64.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextMach_x86_64.cpp @@ -19,7 +19,7 @@ Thread &thread, uint32_t concrete_frame_idx) : RegisterContextDarwin_x86_64(thread, concrete_frame_idx) {} -RegisterContextMach_x86_64::~RegisterContextMach_x86_64() {} +RegisterContextMach_x86_64::~RegisterContextMach_x86_64() = default; int RegisterContextMach_x86_64::DoReadGPR(lldb::tid_t tid, int flavor, GPR &gpr) { diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp @@ -39,7 +39,7 @@ } // Destructor -RegisterContextMemory::~RegisterContextMemory() {} +RegisterContextMemory::~RegisterContextMemory() = default; void RegisterContextMemory::InvalidateAllRegisters() { if (m_reg_data_addr != LLDB_INVALID_ADDRESS) diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm.cpp @@ -45,7 +45,7 @@ : lldb_private::RegisterContext(thread, 0), m_register_info_up(std::move(register_info)) {} -RegisterContextPOSIX_arm::~RegisterContextPOSIX_arm() {} +RegisterContextPOSIX_arm::~RegisterContextPOSIX_arm() = default; void RegisterContextPOSIX_arm::Invalidate() {} diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_arm64.cpp @@ -53,7 +53,7 @@ : lldb_private::RegisterContext(thread, 0), m_register_info_up(std::move(register_info)) {} -RegisterContextPOSIX_arm64::~RegisterContextPOSIX_arm64() {} +RegisterContextPOSIX_arm64::~RegisterContextPOSIX_arm64() = default; void RegisterContextPOSIX_arm64::Invalidate() {} diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_mips64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_mips64.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_mips64.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_mips64.cpp @@ -58,7 +58,7 @@ m_registers_count[msa_registers_count])); } -RegisterContextPOSIX_mips64::~RegisterContextPOSIX_mips64() {} +RegisterContextPOSIX_mips64::~RegisterContextPOSIX_mips64() = default; void RegisterContextPOSIX_mips64::Invalidate() {} diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_powerpc.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_powerpc.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_powerpc.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_powerpc.cpp @@ -95,7 +95,7 @@ m_register_info_up.reset(register_info); } -RegisterContextPOSIX_powerpc::~RegisterContextPOSIX_powerpc() {} +RegisterContextPOSIX_powerpc::~RegisterContextPOSIX_powerpc() = default; void RegisterContextPOSIX_powerpc::Invalidate() {} diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_s390x.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_s390x.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_s390x.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_s390x.cpp @@ -98,7 +98,7 @@ } } -RegisterContextPOSIX_s390x::~RegisterContextPOSIX_s390x() {} +RegisterContextPOSIX_s390x::~RegisterContextPOSIX_s390x() = default; void RegisterContextPOSIX_s390x::Invalidate() {} diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_x86.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_x86.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_x86.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_x86.cpp @@ -448,7 +448,7 @@ m_fpr_type = eNotValid; } -RegisterContextPOSIX_x86::~RegisterContextPOSIX_x86() {} +RegisterContextPOSIX_x86::~RegisterContextPOSIX_x86() = default; RegisterContextPOSIX_x86::FPRType RegisterContextPOSIX_x86::GetFPRType() { if (m_fpr_type == eNotValid) { diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextThreadMemory.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextThreadMemory.cpp --- a/lldb/source/Plugins/Process/Utility/RegisterContextThreadMemory.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextThreadMemory.cpp @@ -22,7 +22,7 @@ : RegisterContext(thread, 0), m_thread_wp(thread.shared_from_this()), m_reg_ctx_sp(), m_register_data_addr(register_data_addr), m_stop_id(0) {} -RegisterContextThreadMemory::~RegisterContextThreadMemory() {} +RegisterContextThreadMemory::~RegisterContextThreadMemory() = default; void RegisterContextThreadMemory::UpdateRegisterContext() { ThreadSP thread_sp(m_thread_wp.lock()); diff --git a/lldb/source/Plugins/Process/Utility/RegisterInfoInterface.h b/lldb/source/Plugins/Process/Utility/RegisterInfoInterface.h --- a/lldb/source/Plugins/Process/Utility/RegisterInfoInterface.h +++ b/lldb/source/Plugins/Process/Utility/RegisterInfoInterface.h @@ -22,7 +22,7 @@ public: RegisterInfoInterface(const lldb_private::ArchSpec &target_arch) : m_target_arch(target_arch) {} - virtual ~RegisterInfoInterface() {} + virtual ~RegisterInfoInterface() = default; virtual size_t GetGPRSize() const = 0; diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm.cpp --- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm.cpp +++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm.cpp @@ -25,7 +25,7 @@ m_gpr.SetByteOrder(gpregset.GetByteOrder()); } -RegisterContextCorePOSIX_arm::~RegisterContextCorePOSIX_arm() {} +RegisterContextCorePOSIX_arm::~RegisterContextCorePOSIX_arm() = default; bool RegisterContextCorePOSIX_arm::ReadGPR() { return true; } diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm64.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm64.cpp --- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm64.cpp +++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm64.cpp @@ -62,7 +62,7 @@ ConfigureRegisterContext(); } -RegisterContextCorePOSIX_arm64::~RegisterContextCorePOSIX_arm64() {} +RegisterContextCorePOSIX_arm64::~RegisterContextCorePOSIX_arm64() = default; bool RegisterContextCorePOSIX_arm64::ReadGPR() { return true; } diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_mips64.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_mips64.cpp --- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_mips64.cpp +++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_mips64.cpp @@ -32,7 +32,7 @@ m_fpr.SetByteOrder(fpregset.GetByteOrder()); } -RegisterContextCorePOSIX_mips64::~RegisterContextCorePOSIX_mips64() {} +RegisterContextCorePOSIX_mips64::~RegisterContextCorePOSIX_mips64() = default; bool RegisterContextCorePOSIX_mips64::ReadGPR() { return true; } diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_powerpc.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_powerpc.cpp --- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_powerpc.cpp +++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_powerpc.cpp @@ -39,7 +39,7 @@ m_vec.SetByteOrder(vregset.GetByteOrder()); } -RegisterContextCorePOSIX_powerpc::~RegisterContextCorePOSIX_powerpc() {} +RegisterContextCorePOSIX_powerpc::~RegisterContextCorePOSIX_powerpc() = default; bool RegisterContextCorePOSIX_powerpc::ReadGPR() { return true; } diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_s390x.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_s390x.cpp --- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_s390x.cpp +++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_s390x.cpp @@ -33,7 +33,7 @@ m_fpr.SetByteOrder(fpregset.GetByteOrder()); } -RegisterContextCorePOSIX_s390x::~RegisterContextCorePOSIX_s390x() {} +RegisterContextCorePOSIX_s390x::~RegisterContextCorePOSIX_s390x() = default; bool RegisterContextCorePOSIX_s390x::ReadGPR() { return true; } diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationHistory.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationHistory.cpp --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationHistory.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationHistory.cpp @@ -24,7 +24,7 @@ m_packets.resize(size); } -GDBRemoteCommunicationHistory::~GDBRemoteCommunicationHistory() {} +GDBRemoteCommunicationHistory::~GDBRemoteCommunicationHistory() = default; void GDBRemoteCommunicationHistory::AddPacket(char packet_char, GDBRemotePacket::Type type, diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp @@ -33,7 +33,7 @@ bool &quit) { return this->Handle_QErrorStringEnable(packet); }); } -GDBRemoteCommunicationServer::~GDBRemoteCommunicationServer() {} +GDBRemoteCommunicationServer::~GDBRemoteCommunicationServer() = default; void GDBRemoteCommunicationServer::RegisterPacketHandler( StringExtractorGDBRemote::ServerPacketType packet_type, diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp @@ -168,7 +168,8 @@ } // Destructor -GDBRemoteCommunicationServerCommon::~GDBRemoteCommunicationServerCommon() {} +GDBRemoteCommunicationServerCommon::~GDBRemoteCommunicationServerCommon() = + default; GDBRemoteCommunication::PacketResult GDBRemoteCommunicationServerCommon::Handle_qHostInfo( diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerPlatform.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerPlatform.cpp --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerPlatform.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerPlatform.cpp @@ -153,7 +153,8 @@ } // Destructor -GDBRemoteCommunicationServerPlatform::~GDBRemoteCommunicationServerPlatform() {} +GDBRemoteCommunicationServerPlatform::~GDBRemoteCommunicationServerPlatform() = + default; Status GDBRemoteCommunicationServerPlatform::LaunchGDBServer( const lldb_private::Args &args, std::string hostname, lldb::pid_t &pid, diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp @@ -50,7 +50,7 @@ } // Destructor -GDBRemoteRegisterContext::~GDBRemoteRegisterContext() {} +GDBRemoteRegisterContext::~GDBRemoteRegisterContext() = default; void GDBRemoteRegisterContext::InvalidateAllRegisters() { SetAllRegisterValid(false); diff --git a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp --- a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp @@ -135,7 +135,7 @@ m_collection_sp->Initialize(g_processgdbremote_properties); } - ~PluginProperties() override {} + ~PluginProperties() override = default; uint64_t GetPacketTimeout() { const uint32_t idx = ePropertyPacketTimeout; @@ -5178,7 +5178,7 @@ m_option_group.Finalize(); } - ~CommandObjectProcessGDBRemoteSpeedTest() override {} + ~CommandObjectProcessGDBRemoteSpeedTest() override = default; Options *GetOptions() override { return &m_option_group; } @@ -5229,7 +5229,7 @@ : CommandObjectParsed(interpreter, "process plugin packet history", "Dumps the packet history buffer. ", nullptr) {} - ~CommandObjectProcessGDBRemotePacketHistory() override {} + ~CommandObjectProcessGDBRemotePacketHistory() override = default; bool DoExecute(Args &command, CommandReturnObject &result) override { const size_t argc = command.GetArgumentCount(); @@ -5260,7 +5260,7 @@ "Maximum size that lldb will try to read/write one one chunk.", nullptr) {} - ~CommandObjectProcessGDBRemotePacketXferSize() override {} + ~CommandObjectProcessGDBRemotePacketXferSize() override = default; bool DoExecute(Args &command, CommandReturnObject &result) override { const size_t argc = command.GetArgumentCount(); @@ -5301,7 +5301,7 @@ "stripped from the result.", nullptr) {} - ~CommandObjectProcessGDBRemotePacketSend() override {} + ~CommandObjectProcessGDBRemotePacketSend() override = default; bool DoExecute(Args &command, CommandReturnObject &result) override { const size_t argc = command.GetArgumentCount(); @@ -5351,7 +5351,7 @@ "encoded into a valid 'qRcmd' packet, sent and the " "response will be printed.") {} - ~CommandObjectProcessGDBRemotePacketMonitor() override {} + ~CommandObjectProcessGDBRemotePacketMonitor() override = default; bool DoExecute(llvm::StringRef command, CommandReturnObject &result) override { @@ -5414,7 +5414,7 @@ interpreter))); } - ~CommandObjectProcessGDBRemotePacket() override {} + ~CommandObjectProcessGDBRemotePacket() override = default; }; class CommandObjectMultiwordProcessGDBRemote : public CommandObjectMultiword { @@ -5429,7 +5429,7 @@ CommandObjectSP(new CommandObjectProcessGDBRemotePacket(interpreter))); } - ~CommandObjectMultiwordProcessGDBRemote() override {} + ~CommandObjectMultiwordProcessGDBRemote() override = default; }; CommandObject *ProcessGDBRemote::GetPluginCommandObject() { diff --git a/lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp b/lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp --- a/lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp +++ b/lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp @@ -871,7 +871,7 @@ m_option_group.Finalize(); } - ~CommandObjectProcessMinidumpDump() override {} + ~CommandObjectProcessMinidumpDump() override = default; Options *GetOptions() override { return &m_option_group; } @@ -1000,7 +1000,7 @@ CommandObjectSP(new CommandObjectProcessMinidumpDump(interpreter))); } - ~CommandObjectMultiwordProcessMinidump() override {} + ~CommandObjectMultiwordProcessMinidump() override = default; }; CommandObject *ProcessMinidump::GetPluginCommandObject() { diff --git a/lldb/source/Plugins/Process/minidump/ThreadMinidump.cpp b/lldb/source/Plugins/Process/minidump/ThreadMinidump.cpp --- a/lldb/source/Plugins/Process/minidump/ThreadMinidump.cpp +++ b/lldb/source/Plugins/Process/minidump/ThreadMinidump.cpp @@ -38,7 +38,7 @@ : Thread(process, td.ThreadId), m_thread_reg_ctx_sp(), m_gpregset_data(gpregset_data) {} -ThreadMinidump::~ThreadMinidump() {} +ThreadMinidump::~ThreadMinidump() = default; void ThreadMinidump::RefreshStateAfterStop() {} diff --git a/lldb/source/Plugins/ScriptInterpreter/Lua/ScriptInterpreterLua.cpp b/lldb/source/Plugins/ScriptInterpreter/Lua/ScriptInterpreterLua.cpp --- a/lldb/source/Plugins/ScriptInterpreter/Lua/ScriptInterpreterLua.cpp +++ b/lldb/source/Plugins/ScriptInterpreter/Lua/ScriptInterpreterLua.cpp @@ -134,7 +134,7 @@ : ScriptInterpreter(debugger, eScriptLanguageLua), m_lua(std::make_unique()) {} -ScriptInterpreterLua::~ScriptInterpreterLua() {} +ScriptInterpreterLua::~ScriptInterpreterLua() = default; bool ScriptInterpreterLua::ExecuteOneLine(llvm::StringRef command, CommandReturnObject *result, diff --git a/lldb/source/Plugins/ScriptInterpreter/None/ScriptInterpreterNone.cpp b/lldb/source/Plugins/ScriptInterpreter/None/ScriptInterpreterNone.cpp --- a/lldb/source/Plugins/ScriptInterpreter/None/ScriptInterpreterNone.cpp +++ b/lldb/source/Plugins/ScriptInterpreter/None/ScriptInterpreterNone.cpp @@ -25,7 +25,7 @@ ScriptInterpreterNone::ScriptInterpreterNone(Debugger &debugger) : ScriptInterpreter(debugger, eScriptLanguageNone) {} -ScriptInterpreterNone::~ScriptInterpreterNone() {} +ScriptInterpreterNone::~ScriptInterpreterNone() = default; bool ScriptInterpreterNone::ExecuteOneLine(llvm::StringRef command, CommandReturnObject *, diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h b/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h --- a/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h +++ b/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h @@ -229,7 +229,7 @@ class PythonObject { public: - PythonObject() {} + PythonObject() = default; PythonObject(PyRefType type, PyObject *py_obj) { m_py_obj = py_obj; @@ -421,7 +421,7 @@ Py_DECREF(py_obj); } - TypedPythonObject() {} + TypedPythonObject() = default; }; class PythonBytes : public TypedPythonObject { diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp --- a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp +++ b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp @@ -1358,7 +1358,7 @@ Status error = ExecuteMultipleLines( function_def_string.c_str(), - ScriptInterpreter::ExecuteScriptOptions().SetEnableIO(false)); + ExecuteScriptOptions().SetEnableIO(false)); return error; } @@ -2767,7 +2767,7 @@ directory.c_str(), directory.c_str()); bool syspath_retval = ExecuteMultipleLines(command_stream.GetData(), - ScriptInterpreter::ExecuteScriptOptions() + ExecuteScriptOptions() .SetEnableIO(false) .SetSetLLDBGlobals(false)) .Success(); @@ -2855,7 +2855,7 @@ (ExecuteOneLineWithReturn( command_stream.GetData(), ScriptInterpreterPythonImpl::eScriptReturnTypeBool, &does_contain, - ScriptInterpreter::ExecuteScriptOptions() + ExecuteScriptOptions() .SetEnableIO(false) .SetSetLLDBGlobals(false)) && does_contain); @@ -2877,7 +2877,7 @@ command_stream.Printf("import %s", module_name.c_str()); error = ExecuteMultipleLines(command_stream.GetData(), - ScriptInterpreter::ExecuteScriptOptions() + ExecuteScriptOptions() .SetEnableIO(false) .SetSetLLDBGlobals(false)); if (error.Fail()) @@ -3055,7 +3055,7 @@ if (ExecuteOneLineWithReturn( command, ScriptInterpreter::eScriptReturnTypeCharStrOrNone, &result_ptr, - ScriptInterpreter::ExecuteScriptOptions().SetEnableIO(false))) { + ExecuteScriptOptions().SetEnableIO(false))) { if (result_ptr) dest.assign(result_ptr); return true; diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h --- a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h +++ b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h @@ -418,7 +418,7 @@ : IOHandler(debugger, IOHandler::Type::PythonInterpreter), m_python(python) {} - ~IOHandlerPythonInterpreter() override {} + ~IOHandlerPythonInterpreter() override = default; ConstString GetControlSequence(char ch) override { if (ch == 'd') diff --git a/lldb/source/Plugins/StructuredData/DarwinLog/StructuredDataDarwinLog.cpp b/lldb/source/Plugins/StructuredData/DarwinLog/StructuredDataDarwinLog.cpp --- a/lldb/source/Plugins/StructuredData/DarwinLog/StructuredDataDarwinLog.cpp +++ b/lldb/source/Plugins/StructuredData/DarwinLog/StructuredDataDarwinLog.cpp @@ -126,7 +126,7 @@ m_collection_sp->Initialize(g_darwinlog_properties); } - ~StructuredDataDarwinLogProperties() override {} + ~StructuredDataDarwinLogProperties() override = default; bool GetEnableOnStartup() const { const uint32_t idx = ePropertyEnableOnStartup; @@ -181,7 +181,7 @@ class FilterRule { public: - virtual ~FilterRule() {} + virtual ~FilterRule() = default; using OperationCreationFunc = std::functionIsAlive() && diff --git a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.h b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.h --- a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.h +++ b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.h @@ -52,9 +52,7 @@ libBacktraceRecording */ uint64_t count = 0; /* the number of queues included in the queues buffer */ - GetQueuesReturnInfo() - - {} + GetQueuesReturnInfo() = default; }; /// Get the list of queues that exist (with any active or pending items) via diff --git a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.cpp b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.cpp --- a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.cpp +++ b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetQueuesHandler.cpp @@ -99,7 +99,7 @@ m_get_queues_return_buffer_addr(LLDB_INVALID_ADDRESS), m_get_queues_retbuffer_mutex() {} -AppleGetQueuesHandler::~AppleGetQueuesHandler() {} +AppleGetQueuesHandler::~AppleGetQueuesHandler() = default; void AppleGetQueuesHandler::Detach() { diff --git a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.h b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.h --- a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.h +++ b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.h @@ -52,7 +52,7 @@ lldb::addr_t item_buffer_size = 0; /* the size of the item buffer from libBacktraceRecording */ - GetThreadItemInfoReturnInfo() {} + GetThreadItemInfoReturnInfo() = default; }; /// Get the information about a work item by calling diff --git a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.cpp b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.cpp --- a/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.cpp +++ b/lldb/source/Plugins/SystemRuntime/MacOSX/AppleGetThreadItemInfoHandler.cpp @@ -107,7 +107,7 @@ m_get_thread_item_info_return_buffer_addr(LLDB_INVALID_ADDRESS), m_get_thread_item_info_retbuffer_mutex() {} -AppleGetThreadItemInfoHandler::~AppleGetThreadItemInfoHandler() {} +AppleGetThreadItemInfoHandler::~AppleGetThreadItemInfoHandler() = default; void AppleGetThreadItemInfoHandler::Detach() { diff --git a/lldb/source/Plugins/SystemRuntime/MacOSX/SystemRuntimeMacOSX.h b/lldb/source/Plugins/SystemRuntime/MacOSX/SystemRuntimeMacOSX.h --- a/lldb/source/Plugins/SystemRuntime/MacOSX/SystemRuntimeMacOSX.h +++ b/lldb/source/Plugins/SystemRuntime/MacOSX/SystemRuntimeMacOSX.h @@ -113,7 +113,7 @@ uint16_t item_info_version = 0; uint16_t item_info_data_offset = 0; - libBacktraceRecording_info() {} + libBacktraceRecording_info() = default; }; // A structure which reflects the data recorded in the @@ -190,9 +190,7 @@ uint16_t vo_activity_ids_array = UINT16_MAX; uint16_t vo_activity_ids_array_entry_size = UINT16_MAX; - LibdispatchVoucherOffsets() - - {} + LibdispatchVoucherOffsets() = default; bool IsValid() { return vo_version != UINT16_MAX; } }; @@ -203,7 +201,7 @@ uint64_t dti_voucher_index = UINT64_MAX; uint64_t dti_qos_class_index = UINT64_MAX; - LibdispatchTSDIndexes() {} + LibdispatchTSDIndexes() = default; bool IsValid() { return dti_version != UINT16_MAX; } }; @@ -214,9 +212,7 @@ uint16_t plo_pthread_tsd_base_address_offset = UINT16_MAX; uint16_t plo_pthread_tsd_entry_size = UINT16_MAX; - LibpthreadOffsets() - - {} + LibpthreadOffsets() = default; bool IsValid() { return plo_version != UINT16_MAX; } }; diff --git a/lldb/source/Plugins/UnwindAssembly/x86/x86AssemblyInspectionEngine.h b/lldb/source/Plugins/UnwindAssembly/x86/x86AssemblyInspectionEngine.h --- a/lldb/source/Plugins/UnwindAssembly/x86/x86AssemblyInspectionEngine.h +++ b/lldb/source/Plugins/UnwindAssembly/x86/x86AssemblyInspectionEngine.h @@ -47,7 +47,7 @@ struct lldb_reg_info { const char *name = nullptr; uint32_t lldb_regnum = LLDB_INVALID_REGNUM; - lldb_reg_info() {} + lldb_reg_info() = default; }; void Initialize(std::vector ®_info); diff --git a/lldb/source/Symbol/ArmUnwindInfo.cpp b/lldb/source/Symbol/ArmUnwindInfo.cpp --- a/lldb/source/Symbol/ArmUnwindInfo.cpp +++ b/lldb/source/Symbol/ArmUnwindInfo.cpp @@ -68,7 +68,7 @@ llvm::sort(m_exidx_entries.begin(), m_exidx_entries.end()); } -ArmUnwindInfo::~ArmUnwindInfo() {} +ArmUnwindInfo::~ArmUnwindInfo() = default; // Read a byte from the unwind instruction stream with the given offset. Custom // function is required because have to red in order of significance within diff --git a/lldb/source/Symbol/Block.cpp b/lldb/source/Symbol/Block.cpp --- a/lldb/source/Symbol/Block.cpp +++ b/lldb/source/Symbol/Block.cpp @@ -25,7 +25,7 @@ m_inlineInfoSP(), m_variable_list_sp(), m_parsed_block_info(false), m_parsed_block_variables(false), m_parsed_child_blocks(false) {} -Block::~Block() {} +Block::~Block() = default; void Block::GetDescription(Stream *s, Function *function, lldb::DescriptionLevel level, Target *target) const { diff --git a/lldb/source/Symbol/CompactUnwindInfo.cpp b/lldb/source/Symbol/CompactUnwindInfo.cpp --- a/lldb/source/Symbol/CompactUnwindInfo.cpp +++ b/lldb/source/Symbol/CompactUnwindInfo.cpp @@ -167,7 +167,7 @@ // destructor -CompactUnwindInfo::~CompactUnwindInfo() {} +CompactUnwindInfo::~CompactUnwindInfo() = default; bool CompactUnwindInfo::GetUnwindPlan(Target &target, Address addr, UnwindPlan &unwind_plan) { diff --git a/lldb/source/Symbol/FuncUnwinders.cpp b/lldb/source/Symbol/FuncUnwinders.cpp --- a/lldb/source/Symbol/FuncUnwinders.cpp +++ b/lldb/source/Symbol/FuncUnwinders.cpp @@ -55,7 +55,7 @@ /// destructor -FuncUnwinders::~FuncUnwinders() {} +FuncUnwinders::~FuncUnwinders() = default; UnwindPlanSP FuncUnwinders::GetUnwindPlanAtCallSite(Target &target, Thread &thread) { diff --git a/lldb/source/Symbol/Function.cpp b/lldb/source/Symbol/Function.cpp --- a/lldb/source/Symbol/Function.cpp +++ b/lldb/source/Symbol/Function.cpp @@ -32,7 +32,7 @@ FunctionInfo::FunctionInfo(ConstString name, const Declaration *decl_ptr) : m_name(name), m_declaration(decl_ptr) {} -FunctionInfo::~FunctionInfo() {} +FunctionInfo::~FunctionInfo() = default; void FunctionInfo::Dump(Stream *s, bool show_fullpaths) const { if (m_name) @@ -74,7 +74,7 @@ : FunctionInfo(name, decl_ptr), m_mangled(mangled), m_call_decl(call_decl_ptr) {} -InlineFunctionInfo::~InlineFunctionInfo() {} +InlineFunctionInfo::~InlineFunctionInfo() = default; void InlineFunctionInfo::Dump(Stream *s, bool show_fullpaths) const { FunctionInfo::Dump(s, show_fullpaths); @@ -238,7 +238,7 @@ assert(comp_unit != nullptr); } -Function::~Function() {} +Function::~Function() = default; void Function::GetStartLineSourceInfo(FileSpec &source_file, uint32_t &line_no) { diff --git a/lldb/source/Symbol/LineTable.cpp b/lldb/source/Symbol/LineTable.cpp --- a/lldb/source/Symbol/LineTable.cpp +++ b/lldb/source/Symbol/LineTable.cpp @@ -34,7 +34,7 @@ } // Destructor -LineTable::~LineTable() {} +LineTable::~LineTable() = default; void LineTable::InsertLineEntry(lldb::addr_t file_addr, uint32_t line, uint16_t column, uint16_t file_idx, @@ -58,7 +58,7 @@ // Dump (&s, Address::DumpStyleFileAddress); } -LineSequence::LineSequence() {} +LineSequence::LineSequence() = default; void LineTable::LineSequenceImpl::Clear() { m_entries.clear(); } diff --git a/lldb/source/Symbol/SymbolContext.cpp b/lldb/source/Symbol/SymbolContext.cpp --- a/lldb/source/Symbol/SymbolContext.cpp +++ b/lldb/source/Symbol/SymbolContext.cpp @@ -51,7 +51,7 @@ sc_scope->CalculateSymbolContext(this); } -SymbolContext::~SymbolContext() {} +SymbolContext::~SymbolContext() = default; void SymbolContext::Clear(bool clear_target) { if (clear_target) @@ -926,7 +926,7 @@ m_start_line(0), m_end_line(0), m_function_spec(), m_class_name(), m_address_range_up(), m_type(eNothingSpecified) {} -SymbolContextSpecifier::~SymbolContextSpecifier() {} +SymbolContextSpecifier::~SymbolContextSpecifier() = default; bool SymbolContextSpecifier::AddLineSpecification(uint32_t line_no, SpecificationType type) { @@ -1186,7 +1186,7 @@ SymbolContextList::SymbolContextList() : m_symbol_contexts() {} -SymbolContextList::~SymbolContextList() {} +SymbolContextList::~SymbolContextList() = default; void SymbolContextList::Append(const SymbolContext &sc) { m_symbol_contexts.push_back(sc); diff --git a/lldb/source/Symbol/Symtab.cpp b/lldb/source/Symbol/Symtab.cpp --- a/lldb/source/Symbol/Symtab.cpp +++ b/lldb/source/Symbol/Symtab.cpp @@ -40,7 +40,7 @@ lldb::eFunctionNameTypeSelector, UniqueCStringMap())); } -Symtab::~Symtab() {} +Symtab::~Symtab() = default; void Symtab::Reserve(size_t count) { // Clients should grab the mutex from this symbol table and lock it manually diff --git a/lldb/source/Symbol/TypeList.cpp b/lldb/source/Symbol/TypeList.cpp --- a/lldb/source/Symbol/TypeList.cpp +++ b/lldb/source/Symbol/TypeList.cpp @@ -22,7 +22,7 @@ TypeList::TypeList() : m_types() {} // Destructor -TypeList::~TypeList() {} +TypeList::~TypeList() = default; void TypeList::Insert(const TypeSP &type_sp) { // Just push each type on the back for now. We will worry about uniquing diff --git a/lldb/source/Symbol/TypeMap.cpp b/lldb/source/Symbol/TypeMap.cpp --- a/lldb/source/Symbol/TypeMap.cpp +++ b/lldb/source/Symbol/TypeMap.cpp @@ -22,7 +22,7 @@ TypeMap::TypeMap() : m_types() {} // Destructor -TypeMap::~TypeMap() {} +TypeMap::~TypeMap() = default; void TypeMap::Insert(const TypeSP &type_sp) { // Just push each type on the back for now. We will worry about uniquing diff --git a/lldb/source/Symbol/TypeSystem.cpp b/lldb/source/Symbol/TypeSystem.cpp --- a/lldb/source/Symbol/TypeSystem.cpp +++ b/lldb/source/Symbol/TypeSystem.cpp @@ -35,7 +35,7 @@ bool LanguageSet::Empty() const { return bitvector.none(); } bool LanguageSet::operator[](unsigned i) const { return bitvector[i]; } -TypeSystem::~TypeSystem() {} +TypeSystem::~TypeSystem() = default; static lldb::TypeSystemSP CreateInstanceHelper(lldb::LanguageType language, Module *module, Target *target) { @@ -180,7 +180,7 @@ TypeSystemMap::TypeSystemMap() : m_mutex(), m_map() {} -TypeSystemMap::~TypeSystemMap() {} +TypeSystemMap::~TypeSystemMap() = default; void TypeSystemMap::Clear() { collection map; diff --git a/lldb/source/Symbol/UnwindTable.cpp b/lldb/source/Symbol/UnwindTable.cpp --- a/lldb/source/Symbol/UnwindTable.cpp +++ b/lldb/source/Symbol/UnwindTable.cpp @@ -83,7 +83,7 @@ } } -UnwindTable::~UnwindTable() {} +UnwindTable::~UnwindTable() = default; llvm::Optional UnwindTable::GetAddressRange(const Address &addr, SymbolContext &sc) { diff --git a/lldb/source/Symbol/Variable.cpp b/lldb/source/Symbol/Variable.cpp --- a/lldb/source/Symbol/Variable.cpp +++ b/lldb/source/Symbol/Variable.cpp @@ -49,7 +49,7 @@ m_artificial(artificial), m_loc_is_const_data(location_is_constant_data), m_static_member(static_member) {} -Variable::~Variable() {} +Variable::~Variable() = default; lldb::LanguageType Variable::GetLanguage() const { lldb::LanguageType lang = m_mangled.GuessLanguage(); diff --git a/lldb/source/Symbol/VariableList.cpp b/lldb/source/Symbol/VariableList.cpp --- a/lldb/source/Symbol/VariableList.cpp +++ b/lldb/source/Symbol/VariableList.cpp @@ -20,7 +20,7 @@ VariableList::VariableList() : m_variables() {} // Destructor -VariableList::~VariableList() {} +VariableList::~VariableList() = default; void VariableList::AddVariable(const VariableSP &var_sp) { m_variables.push_back(var_sp); diff --git a/lldb/source/Target/JITLoaderList.cpp b/lldb/source/Target/JITLoaderList.cpp --- a/lldb/source/Target/JITLoaderList.cpp +++ b/lldb/source/Target/JITLoaderList.cpp @@ -15,7 +15,7 @@ JITLoaderList::JITLoaderList() : m_jit_loaders_vec(), m_jit_loaders_mutex() {} -JITLoaderList::~JITLoaderList() {} +JITLoaderList::~JITLoaderList() = default; void JITLoaderList::Append(const JITLoaderSP &jit_loader_sp) { std::lock_guard guard(m_jit_loaders_mutex); diff --git a/lldb/source/Target/Language.cpp b/lldb/source/Target/Language.cpp --- a/lldb/source/Target/Language.cpp +++ b/lldb/source/Target/Language.cpp @@ -448,7 +448,7 @@ catch_on ? "on" : "off", throw_on ? "on" : "off"); } // Constructor -Language::Language() {} +Language::Language() = default; // Destructor -Language::~Language() {} +Language::~Language() = default; diff --git a/lldb/source/Target/Memory.cpp b/lldb/source/Target/Memory.cpp --- a/lldb/source/Target/Memory.cpp +++ b/lldb/source/Target/Memory.cpp @@ -26,7 +26,7 @@ m_L2_cache_line_byte_size(process.GetMemoryCacheLineSize()) {} // Destructor -MemoryCache::~MemoryCache() {} +MemoryCache::~MemoryCache() = default; void MemoryCache::Clear(bool clear_invalid_ranges) { std::lock_guard guard(m_mutex); @@ -259,7 +259,7 @@ assert(byte_size > chunk_size); } -AllocatedBlock::~AllocatedBlock() {} +AllocatedBlock::~AllocatedBlock() = default; lldb::addr_t AllocatedBlock::ReserveBlock(uint32_t size) { // We must return something valid for zero bytes. @@ -329,7 +329,7 @@ AllocatedMemoryCache::AllocatedMemoryCache(Process &process) : m_process(process), m_mutex(), m_memory_map() {} -AllocatedMemoryCache::~AllocatedMemoryCache() {} +AllocatedMemoryCache::~AllocatedMemoryCache() = default; void AllocatedMemoryCache::Clear() { std::lock_guard guard(m_mutex); diff --git a/lldb/source/Target/QueueItem.cpp b/lldb/source/Target/QueueItem.cpp --- a/lldb/source/Target/QueueItem.cpp +++ b/lldb/source/Target/QueueItem.cpp @@ -27,7 +27,7 @@ m_process_wp = process_sp; } -QueueItem::~QueueItem() {} +QueueItem::~QueueItem() = default; QueueItemKind QueueItem::GetKind() { FetchEntireItem(); diff --git a/lldb/source/Target/StructuredDataPlugin.cpp b/lldb/source/Target/StructuredDataPlugin.cpp --- a/lldb/source/Target/StructuredDataPlugin.cpp +++ b/lldb/source/Target/StructuredDataPlugin.cpp @@ -23,14 +23,14 @@ "Parent for per-plugin structured data commands", "plugin structured-data ") {} - ~CommandStructuredData() override {} + ~CommandStructuredData() override = default; }; } StructuredDataPlugin::StructuredDataPlugin(const ProcessWP &process_wp) : PluginInterface(), m_process_wp(process_wp) {} -StructuredDataPlugin::~StructuredDataPlugin() {} +StructuredDataPlugin::~StructuredDataPlugin() = default; bool StructuredDataPlugin::GetEnabled(ConstString type_name) const { // By default, plugins are always enabled. Plugin authors should override diff --git a/lldb/source/Target/ThreadPlanBase.cpp b/lldb/source/Target/ThreadPlanBase.cpp --- a/lldb/source/Target/ThreadPlanBase.cpp +++ b/lldb/source/Target/ThreadPlanBase.cpp @@ -43,7 +43,7 @@ SetIsMasterPlan(true); } -ThreadPlanBase::~ThreadPlanBase() {} +ThreadPlanBase::~ThreadPlanBase() = default; void ThreadPlanBase::GetDescription(Stream *s, lldb::DescriptionLevel level) { s->Printf("Base thread plan."); diff --git a/lldb/source/Target/ThreadPlanCallUserExpression.cpp b/lldb/source/Target/ThreadPlanCallUserExpression.cpp --- a/lldb/source/Target/ThreadPlanCallUserExpression.cpp +++ b/lldb/source/Target/ThreadPlanCallUserExpression.cpp @@ -43,7 +43,7 @@ SetOkayToDiscard(false); } -ThreadPlanCallUserExpression::~ThreadPlanCallUserExpression() {} +ThreadPlanCallUserExpression::~ThreadPlanCallUserExpression() = default; void ThreadPlanCallUserExpression::GetDescription( Stream *s, lldb::DescriptionLevel level) { diff --git a/lldb/source/Target/ThreadPlanStepOverBreakpoint.cpp b/lldb/source/Target/ThreadPlanStepOverBreakpoint.cpp --- a/lldb/source/Target/ThreadPlanStepOverBreakpoint.cpp +++ b/lldb/source/Target/ThreadPlanStepOverBreakpoint.cpp @@ -36,7 +36,7 @@ m_breakpoint_addr); } -ThreadPlanStepOverBreakpoint::~ThreadPlanStepOverBreakpoint() {} +ThreadPlanStepOverBreakpoint::~ThreadPlanStepOverBreakpoint() = default; void ThreadPlanStepOverBreakpoint::GetDescription( Stream *s, lldb::DescriptionLevel level) { diff --git a/lldb/source/Utility/ArchSpec.cpp b/lldb/source/Utility/ArchSpec.cpp --- a/lldb/source/Utility/ArchSpec.cpp +++ b/lldb/source/Utility/ArchSpec.cpp @@ -507,7 +507,7 @@ //===----------------------------------------------------------------------===// // Constructors and destructors. -ArchSpec::ArchSpec() {} +ArchSpec::ArchSpec() = default; ArchSpec::ArchSpec(const char *triple_cstr) { if (triple_cstr) diff --git a/lldb/source/Utility/Args.cpp b/lldb/source/Utility/Args.cpp --- a/lldb/source/Utility/Args.cpp +++ b/lldb/source/Utility/Args.cpp @@ -194,7 +194,7 @@ } // Destructor -Args::~Args() {} +Args::~Args() = default; void Args::Dump(Stream &s, const char *label_name) const { if (!label_name) diff --git a/lldb/source/Utility/DataBufferLLVM.cpp b/lldb/source/Utility/DataBufferLLVM.cpp --- a/lldb/source/Utility/DataBufferLLVM.cpp +++ b/lldb/source/Utility/DataBufferLLVM.cpp @@ -21,7 +21,7 @@ "Cannot construct a DataBufferLLVM with a null buffer"); } -DataBufferLLVM::~DataBufferLLVM() {} +DataBufferLLVM::~DataBufferLLVM() = default; uint8_t *DataBufferLLVM::GetBytes() { return reinterpret_cast(Buffer->getBufferStart()); diff --git a/lldb/source/Utility/Event.cpp b/lldb/source/Utility/Event.cpp --- a/lldb/source/Utility/Event.cpp +++ b/lldb/source/Utility/Event.cpp @@ -198,7 +198,7 @@ : EventData(), m_process_sp(process_sp), m_object_sp(object_sp), m_plugin_sp(plugin_sp) {} -EventDataStructuredData::~EventDataStructuredData() {} +EventDataStructuredData::~EventDataStructuredData() = default; // EventDataStructuredData member functions diff --git a/lldb/source/Utility/GDBRemote.cpp b/lldb/source/Utility/GDBRemote.cpp --- a/lldb/source/Utility/GDBRemote.cpp +++ b/lldb/source/Utility/GDBRemote.cpp @@ -24,7 +24,7 @@ ByteOrder byte_order) : StreamString(flags, addr_size, byte_order) {} -StreamGDBRemote::~StreamGDBRemote() {} +StreamGDBRemote::~StreamGDBRemote() = default; int StreamGDBRemote::PutEscapedBytes(const void *s, size_t src_len) { int bytes_written = 0; diff --git a/lldb/source/Utility/Stream.cpp b/lldb/source/Utility/Stream.cpp --- a/lldb/source/Utility/Stream.cpp +++ b/lldb/source/Utility/Stream.cpp @@ -32,7 +32,7 @@ m_forwarder(*this, colors) {} // Destructor -Stream::~Stream() {} +Stream::~Stream() = default; ByteOrder Stream::SetByteOrder(ByteOrder byte_order) { ByteOrder old_byte_order = m_byte_order; diff --git a/lldb/source/Utility/StreamString.cpp b/lldb/source/Utility/StreamString.cpp --- a/lldb/source/Utility/StreamString.cpp +++ b/lldb/source/Utility/StreamString.cpp @@ -17,7 +17,7 @@ ByteOrder byte_order) : Stream(flags, addr_size, byte_order), m_packet() {} -StreamString::~StreamString() {} +StreamString::~StreamString() = default; void StreamString::Flush() { // Nothing to do when flushing a buffer based stream... diff --git a/lldb/source/Utility/StringExtractor.cpp b/lldb/source/Utility/StringExtractor.cpp --- a/lldb/source/Utility/StringExtractor.cpp +++ b/lldb/source/Utility/StringExtractor.cpp @@ -40,7 +40,7 @@ } // Destructor -StringExtractor::~StringExtractor() {} +StringExtractor::~StringExtractor() = default; char StringExtractor::GetChar(char fail_value) { if (m_index < m_packet.size()) { diff --git a/lldb/source/Utility/StringList.cpp b/lldb/source/Utility/StringList.cpp --- a/lldb/source/Utility/StringList.cpp +++ b/lldb/source/Utility/StringList.cpp @@ -33,7 +33,7 @@ } } -StringList::~StringList() {} +StringList::~StringList() = default; void StringList::AppendString(const char *str) { if (str) diff --git a/lldb/source/Utility/TildeExpressionResolver.cpp b/lldb/source/Utility/TildeExpressionResolver.cpp --- a/lldb/source/Utility/TildeExpressionResolver.cpp +++ b/lldb/source/Utility/TildeExpressionResolver.cpp @@ -27,7 +27,7 @@ namespace fs = llvm::sys::fs; namespace path = llvm::sys::path; -TildeExpressionResolver::~TildeExpressionResolver() {} +TildeExpressionResolver::~TildeExpressionResolver() = default; bool StandardTildeExpressionResolver::ResolveExact( StringRef Expr, SmallVectorImpl &Output) {