From 6aa60b0514865751ea9dd208236db60eb69aaf1e Mon Sep 17 00:00:00 2001 From: Pavel Labath Date: Tue, 12 Nov 2019 14:25:18 +0100 Subject: [PATCH] [lldb] Fix more -Wdeprecated-copy warnings This warning triggers when a class defines a copy constructor but not a copy-assignment operator (which then gets auto-generated by the compiler). Fix the warning by deleting the other operator too, as the default implementation works just fine. --- lldb/include/lldb/Core/SearchFilter.h | 7 ------- lldb/include/lldb/Host/SocketAddress.h | 2 -- lldb/include/lldb/Utility/StringExtractorGDBRemote.h | 5 ----- lldb/source/Core/SearchFilter.cpp | 20 -------------------- lldb/source/Host/common/SocketAddress.cpp | 6 ------ lldb/source/Host/common/TCPSocket.cpp | 8 ++++---- 6 files changed, 4 insertions(+), 44 deletions(-) diff --git a/lldb/include/lldb/Core/SearchFilter.h b/lldb/include/lldb/Core/SearchFilter.h index 6823daf..97880f6 100644 --- a/lldb/include/lldb/Core/SearchFilter.h +++ b/lldb/include/lldb/Core/SearchFilter.h @@ -366,8 +366,6 @@ public: ~SearchFilterByModuleList() override; - SearchFilterByModuleList &operator=(const SearchFilterByModuleList &rhs); - bool ModulePasses(const lldb::ModuleSP &module_sp) override; bool ModulePasses(const FileSpec &spec) override; @@ -416,13 +414,8 @@ public: const FileSpecList &module_list, const FileSpecList &cu_list); - SearchFilterByModuleListAndCU(const SearchFilterByModuleListAndCU &rhs); - ~SearchFilterByModuleListAndCU() override; - SearchFilterByModuleListAndCU & - operator=(const SearchFilterByModuleListAndCU &rhs); - bool AddressPasses(Address &address) override; bool CompUnitPasses(FileSpec &fileSpec) override; diff --git a/lldb/include/lldb/Host/SocketAddress.h b/lldb/include/lldb/Host/SocketAddress.h index 620827f..766303a 100644 --- a/lldb/include/lldb/Host/SocketAddress.h +++ b/lldb/include/lldb/Host/SocketAddress.h @@ -48,8 +48,6 @@ public: ~SocketAddress(); // Operators - const SocketAddress &operator=(const SocketAddress &rhs); - const SocketAddress &operator=(const struct addrinfo *addr_info); const SocketAddress &operator=(const struct sockaddr &s); diff --git a/lldb/include/lldb/Utility/StringExtractorGDBRemote.h b/lldb/include/lldb/Utility/StringExtractorGDBRemote.h index cbf6e0c..715f3cb 100644 --- a/lldb/include/lldb/Utility/StringExtractorGDBRemote.h +++ b/lldb/include/lldb/Utility/StringExtractorGDBRemote.h @@ -31,11 +31,6 @@ public: StringExtractorGDBRemote(const char *cstr) : StringExtractor(cstr), m_validator(nullptr) {} - StringExtractorGDBRemote(const StringExtractorGDBRemote &rhs) - : StringExtractor(rhs), m_validator(rhs.m_validator) {} - - ~StringExtractorGDBRemote() override {} - bool ValidateResponse() const; void CopyResponseValidator(const StringExtractorGDBRemote &rhs); diff --git a/lldb/source/Core/SearchFilter.cpp b/lldb/source/Core/SearchFilter.cpp index e02b4f6..8f80caa 100644 --- a/lldb/source/Core/SearchFilter.cpp +++ b/lldb/source/Core/SearchFilter.cpp @@ -523,13 +523,6 @@ SearchFilterByModuleList::SearchFilterByModuleList( enum FilterTy filter_ty) : SearchFilter(target_sp, filter_ty), m_module_spec_list(module_list) {} -SearchFilterByModuleList &SearchFilterByModuleList:: -operator=(const SearchFilterByModuleList &rhs) { - m_target_sp = rhs.m_target_sp; - m_module_spec_list = rhs.m_module_spec_list; - return *this; -} - SearchFilterByModuleList::~SearchFilterByModuleList() = default; bool SearchFilterByModuleList::ModulePasses(const ModuleSP &module_sp) { @@ -669,19 +662,6 @@ SearchFilterByModuleListAndCU::SearchFilterByModuleListAndCU( FilterTy::ByModulesAndCU), m_cu_spec_list(cu_list) {} -SearchFilterByModuleListAndCU::SearchFilterByModuleListAndCU( - const SearchFilterByModuleListAndCU &rhs) = default; - -SearchFilterByModuleListAndCU &SearchFilterByModuleListAndCU:: -operator=(const SearchFilterByModuleListAndCU &rhs) { - if (&rhs != this) { - m_target_sp = rhs.m_target_sp; - m_module_spec_list = rhs.m_module_spec_list; - m_cu_spec_list = rhs.m_cu_spec_list; - } - return *this; -} - SearchFilterByModuleListAndCU::~SearchFilterByModuleListAndCU() = default; lldb::SearchFilterSP SearchFilterByModuleListAndCU::CreateFromStructuredData( diff --git a/lldb/source/Host/common/SocketAddress.cpp b/lldb/source/Host/common/SocketAddress.cpp index 882fd24..960ed18 100644 --- a/lldb/source/Host/common/SocketAddress.cpp +++ b/lldb/source/Host/common/SocketAddress.cpp @@ -174,12 +174,6 @@ bool SocketAddress::SetPort(uint16_t port) { } // SocketAddress assignment operator -const SocketAddress &SocketAddress::operator=(const SocketAddress &rhs) { - if (this != &rhs) - m_socket_addr = rhs.m_socket_addr; - return *this; -} - const SocketAddress &SocketAddress:: operator=(const struct addrinfo *addr_info) { Clear(); diff --git a/lldb/source/Host/common/TCPSocket.cpp b/lldb/source/Host/common/TCPSocket.cpp index e84054f..8be0b2e 100644 --- a/lldb/source/Host/common/TCPSocket.cpp +++ b/lldb/source/Host/common/TCPSocket.cpp @@ -149,9 +149,9 @@ Status TCPSocket::Connect(llvm::StringRef name) { if (!DecodeHostAndPort(name, host_str, port_str, port, &error)) return error; - auto addresses = lldb_private::SocketAddress::GetAddressInfo( + std::vector addresses = SocketAddress::GetAddressInfo( host_str.c_str(), nullptr, AF_UNSPEC, SOCK_STREAM, IPPROTO_TCP); - for (auto address : addresses) { + for (SocketAddress &address : addresses) { error = CreateSocket(address.GetFamily()); if (error.Fail()) continue; @@ -187,9 +187,9 @@ Status TCPSocket::Listen(llvm::StringRef name, int backlog) { if (host_str == "*") host_str = "0.0.0.0"; - auto addresses = lldb_private::SocketAddress::GetAddressInfo( + std::vector addresses = SocketAddress::GetAddressInfo( host_str.c_str(), nullptr, AF_UNSPEC, SOCK_STREAM, IPPROTO_TCP); - for (auto address : addresses) { + for (SocketAddress &address : addresses) { int fd = Socket::CreateSocket(address.GetFamily(), kType, IPPROTO_TCP, m_child_processes_inherit, error); if (error.Fail()) { -- 2.7.4