[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-30 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG4cf9d1e4492f: [lldb][NFC] Modernize for-loops in ModuleList (authored by teemperor). Herald added a subscriber: lldb-commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://rev

[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib accepted this revision as: mib. mib added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112379/new/ https://reviews.llvm.org/D112379 ___ lldb-commits mailing list lldb-commi

[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor added a comment. Straightforward but looking for a second set of eyes Comment at: lldb/source/Core/ModuleList.cpp:204 +bool ModuleList::AppendIfNeeded(const ModuleSP &new_module, bool notify) { + if (new_module) { std::lock_guard guard(m_modules_mutex);

[Lldb-commits] [PATCH] D112379: [lldb][NFC] Modernize for-loops in ModuleList

2021-10-24 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. teemperor added a reviewer: LLDB. teemperor added a project: LLDB. Herald added a subscriber: JDevlieghere. teemperor requested review of this revision. https://reviews.llvm.org/D112379 Files: lldb/include/lldb/Core/ModuleList.h lldb/source/Core/ModuleList.cpp