diff --git a/lldb/include/lldb/Interpreter/Interfaces/ScriptedPlatformInterface.h b/lldb/include/lldb/Interpreter/Interfaces/ScriptedPlatformInterface.h index 2dcbb47ffa6de8..7feaa01fe89b86 100644 --- a/lldb/include/lldb/Interpreter/Interfaces/ScriptedPlatformInterface.h +++ b/lldb/include/lldb/Interpreter/Interfaces/ScriptedPlatformInterface.h @@ -22,9 +22,7 @@ class ScriptedPlatformInterface : virtual public ScriptedInterface { virtual llvm::Expected CreatePluginObject(llvm::StringRef class_name, ExecutionContext &exe_ctx, StructuredData::DictionarySP args_sp, - StructuredData::Generic *script_obj = nullptr) { - return {llvm::make_error()}; - } + StructuredData::Generic *script_obj = nullptr) = 0; virtual StructuredData::DictionarySP ListProcesses() { return {}; } diff --git a/lldb/include/lldb/Interpreter/Interfaces/ScriptedProcessInterface.h b/lldb/include/lldb/Interpreter/Interfaces/ScriptedProcessInterface.h index a429cacd862f12..10203b1f8baa7a 100644 --- a/lldb/include/lldb/Interpreter/Interfaces/ScriptedProcessInterface.h +++ b/lldb/include/lldb/Interpreter/Interfaces/ScriptedProcessInterface.h @@ -24,9 +24,7 @@ class ScriptedProcessInterface : virtual public ScriptedInterface { virtual llvm::Expected CreatePluginObject(llvm::StringRef class_name, ExecutionContext &exe_ctx, StructuredData::DictionarySP args_sp, - StructuredData::Generic *script_obj = nullptr) { - return {llvm::make_error()}; - } + StructuredData::Generic *script_obj = nullptr) = 0; virtual StructuredData::DictionarySP GetCapabilities() { return {}; } diff --git a/lldb/include/lldb/Interpreter/Interfaces/ScriptedThreadInterface.h b/lldb/include/lldb/Interpreter/Interfaces/ScriptedThreadInterface.h index 107e593b5561ef..a7cfc690b67dc7 100644 --- a/lldb/include/lldb/Interpreter/Interfaces/ScriptedThreadInterface.h +++ b/lldb/include/lldb/Interpreter/Interfaces/ScriptedThreadInterface.h @@ -23,9 +23,7 @@ class ScriptedThreadInterface : virtual public ScriptedInterface { virtual llvm::Expected CreatePluginObject(llvm::StringRef class_name, ExecutionContext &exe_ctx, StructuredData::DictionarySP args_sp, - StructuredData::Generic *script_obj = nullptr) { - return {llvm::make_error()}; - } + StructuredData::Generic *script_obj = nullptr) = 0; virtual lldb::tid_t GetThreadID() { return LLDB_INVALID_THREAD_ID; } diff --git a/lldb/include/lldb/Interpreter/ScriptInterpreter.h b/lldb/include/lldb/Interpreter/ScriptInterpreter.h index 0146eeb8626200..b941f6012a117b 100644 --- a/lldb/include/lldb/Interpreter/ScriptInterpreter.h +++ b/lldb/include/lldb/Interpreter/ScriptInterpreter.h @@ -151,10 +151,7 @@ class ScriptInterpreter : public PluginInterface { eScriptReturnTypeOpaqueObject }; - ScriptInterpreter( - Debugger &debugger, lldb::ScriptLanguage script_lang, - lldb::ScriptedPlatformInterfaceUP scripted_platform_interface_up = - std::make_unique()); + ScriptInterpreter(Debugger &debugger, lldb::ScriptLanguage script_lang); virtual StructuredData::DictionarySP GetInterpreterInfo(); @@ -559,19 +556,19 @@ class ScriptInterpreter : public PluginInterface { lldb::ScriptLanguage GetLanguage() { return m_script_lang; } virtual lldb::ScriptedProcessInterfaceUP CreateScriptedProcessInterface() { - return std::make_unique(); + return {}; } virtual lldb::ScriptedThreadInterfaceSP CreateScriptedThreadInterface() { - return std::make_shared(); + return {}; } virtual lldb::OperatingSystemInterfaceSP CreateOperatingSystemInterface() { - return std::make_shared(); + return {}; } - ScriptedPlatformInterface &GetScriptedPlatformInterface() { - return *m_scripted_platform_interface_up; + virtual lldb::ScriptedPlatformInterfaceUP GetScriptedPlatformInterface() { + return {}; } virtual StructuredData::ObjectSP @@ -599,7 +596,6 @@ class ScriptInterpreter : public PluginInterface { protected: Debugger &m_debugger; lldb::ScriptLanguage m_script_lang; - lldb::ScriptedPlatformInterfaceUP m_scripted_platform_interface_up; }; } // namespace lldb_private diff --git a/lldb/source/Interpreter/ScriptInterpreter.cpp b/lldb/source/Interpreter/ScriptInterpreter.cpp index fb3fa74d0b9780..8dd499ce819a78 100644 --- a/lldb/source/Interpreter/ScriptInterpreter.cpp +++ b/lldb/source/Interpreter/ScriptInterpreter.cpp @@ -27,12 +27,9 @@ using namespace lldb; using namespace lldb_private; -ScriptInterpreter::ScriptInterpreter( - Debugger &debugger, lldb::ScriptLanguage script_lang, - lldb::ScriptedPlatformInterfaceUP scripted_platform_interface_up) - : m_debugger(debugger), m_script_lang(script_lang), - m_scripted_platform_interface_up( - std::move(scripted_platform_interface_up)) {} +ScriptInterpreter::ScriptInterpreter(Debugger &debugger, + lldb::ScriptLanguage script_lang) + : m_debugger(debugger), m_script_lang(script_lang) {} void ScriptInterpreter::CollectDataForBreakpointCommandCallback( std::vector> &bp_options_vec, diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp index 953f8b3aba18f7..ef7a2c128a2207 100644 --- a/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp +++ b/lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp @@ -427,8 +427,6 @@ ScriptInterpreterPythonImpl::ScriptInterpreterPythonImpl(Debugger &debugger) m_active_io_handler(eIOHandlerNone), m_session_is_active(false), m_pty_secondary_is_open(false), m_valid_session(true), m_lock_count(0), m_command_thread_state(nullptr) { - m_scripted_platform_interface_up = - std::make_unique(*this); m_dictionary_name.append("_dict"); StreamString run_string;