[Lldb-commits] [lldb] [lldb/script] Add scripting extension template generator (PR #209647)

[Jonas Devlieghere via lldb-commits]

================
@@ -41,7 +42,15 @@ class ScriptInterpreterPython : public ScriptInterpreter,
       : ScriptInterpreter(debugger, lldb::eScriptLanguagePython),
         IOHandlerDelegateMultiline("DONE") {}
 
+  llvm::Expected<std::string>
+  ExtensionToImportPath(lldb::ScriptedExtension extension) override;
   StructuredData::DictionarySP GetInterpreterInfo() override;
+  llvm::Expected<FileSpec> GenerateExtensionTemplate(
+      const std::string name,
+      std::vector<std::pair<llvm::StringRef,
----------------
JDevlieghere wrote:

Can this be const? Also, this seems like it could benefit from a named struct compared to a plain pair. I think it's also worth documenting the StringRef lifetimes. 

https://github.com/llvm/llvm-project/pull/209647