rnk created this revision. rnk added reviewers: JDevlieghere, amccarth. Herald added a project: LLDB.
This appears to be a real bug caught by -Wunused-value. std::find_if doesn't modify the underlying collection, it just returns an iterator pointing to the matching element. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D74010 Files: lldb/source/Utility/Broadcaster.cpp Index: lldb/source/Utility/Broadcaster.cpp =================================================================== --- lldb/source/Utility/Broadcaster.cpp +++ lldb/source/Utility/Broadcaster.cpp @@ -406,7 +406,7 @@ listener_collection::iterator iter = m_listeners.begin(), end_iter = m_listeners.end(); - std::find_if(iter, end_iter, predicate); + iter = std::find_if(iter, end_iter, predicate); if (iter != end_iter) m_listeners.erase(iter);
Index: lldb/source/Utility/Broadcaster.cpp =================================================================== --- lldb/source/Utility/Broadcaster.cpp +++ lldb/source/Utility/Broadcaster.cpp @@ -406,7 +406,7 @@ listener_collection::iterator iter = m_listeners.begin(), end_iter = m_listeners.end(); - std::find_if(iter, end_iter, predicate); + iter = std::find_if(iter, end_iter, predicate); if (iter != end_iter) m_listeners.erase(iter);
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits