[Lldb-commits] [lldb] [lldb] Unify Platform::ResolveExecutable (PR #96256)

[Jonas Devlieghere via lldb-commits]

================
@@ -1004,10 +1004,7 @@ class Platform : public PluginInterface {
 
   virtual const char *GetCacheHostname();
 
-  virtual Status
-  ResolveRemoteExecutable(const ModuleSpec &module_spec,
-                          lldb::ModuleSP &exe_module_sp,
-                          const FileSpecList *module_search_paths_ptr);
+  /// The method is virtual for mocking in the unit tests.
----------------
JDevlieghere wrote:

This comment is outdated. It was for `ResolveRemoteExecutable` which was virtual but only overridden in the unit test. I'll remove it. 

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