jingham added a comment.

In D61921#1502502 <https://reviews.llvm.org/D61921#1502502>, @labath wrote:

> If this iteration is going to be used a lot, I'd recommend taking a bit of 
> time to implement an iterator abstraction over the language runtimes. It 
> takes a bit longer to set up, but I hope we can all agree that `for (runtime: 
> process->GetLanguageRuntimes()) runtime->foo();` is more readable than 
> `process->ForEachLanguageRuntime([] (runtime) { runtime->foo(); })`. This is 
> particularly true if you need some sort of a control flow construct 
> (`continue`, `break`, `return`) in the loop "body".


+1


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

https://reviews.llvm.org/D61921



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to