diff options
author | Pavel Labath <labath@google.com> | 2017-02-02 17:50:03 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2017-02-02 17:50:03 +0000 |
commit | 97f3c2090d61b9072e4ebb3c966533a0c325c55f (patch) | |
tree | 335afc38db3448f7dc6ec8b1a7278b63f77498d8 /source/Plugins/Platform/NetBSD/PlatformNetBSD.h | |
parent | 6e129902710eb581d7902f74e82e4687a752f0a7 (diff) |
Unify PlatformPOSIX::ResolveExecutable
Summary:
various platforms very using nearly identical code for this method. As far as I
can tell there was nothing platform-specific about the differences, but rather
it looked like it was caused by tiny tweaks being made to individual copies
without affecting the overall functionality. I have taken the superset of all
these tweaks and put it into one method in the base class (incidentaly, nobody
was using the base class implementation of the method, as all classes were
overriding it). From the darwin class I took the slightly improved error
reporting (checking whether the file is readable) and the
ResolveExecutableInBundle call (which has no effect elsewhere as the function
is already a no-op on non-darwin platforms). From the linux class I took the
set-the-triple-vendor-to-host-if-unspecified tweak (present in PlatformKalimba
as well).
PlatformWindows has an identical copy as well. We could resolve that by pushing
this code further down into Platform class, that that would require pushing the
m_remote_platform_sp member as well, which seems like a bad design choice.
Reviewers: clayborg, emaste, krytarowski
Subscribers: lldb-commits
Differential Revision: https://reviews.llvm.org/D29406
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@293910 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'source/Plugins/Platform/NetBSD/PlatformNetBSD.h')
-rw-r--r-- | source/Plugins/Platform/NetBSD/PlatformNetBSD.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/source/Plugins/Platform/NetBSD/PlatformNetBSD.h b/source/Plugins/Platform/NetBSD/PlatformNetBSD.h index ddca5eb5b..53c76f28f 100644 --- a/source/Plugins/Platform/NetBSD/PlatformNetBSD.h +++ b/source/Plugins/Platform/NetBSD/PlatformNetBSD.h @@ -60,10 +60,6 @@ public: std::string *command_output, uint32_t timeout_sec) override; - Error ResolveExecutable(const ModuleSpec &module_spec, - lldb::ModuleSP &module_sp, - const FileSpecList *module_search_paths_ptr) override; - bool GetRemoteOSVersion() override; bool GetRemoteOSBuildString(std::string &s) override; |