[Lldb-commits] [PATCH] D139083: [lldb][Module][NFC] Add ModuleList::AnyOf

[Michael Buch via Phabricator via lldb-commits]

Michael137 added inline comments.


================
Comment at: lldb/include/lldb/Core/ModuleList.h:480
+  /// This function is thread-safe.
+  bool AnyOf(std::function<bool(const lldb::ModuleSP &module_sp)> const
+                 &callback) const;
----------------
aprantl wrote:
> Why not `std::function<bool(const lldb::Module &)>` ?
Unfortunately a lot of APIs of `Module` are non-const since they rely on updating internal state :(


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139083/new/

https://reviews.llvm.org/D139083