[PATCH] D54861: [SanitizerCommon] Test `CombinedAllocator::ForEachChunk()` in unit tests.

Mon Nov 26 12:18:04 PST 2018

delcypher added a comment.

In D54861#1308171 <https://reviews.llvm.org/D54861#1308171>, @kubamracek wrote:

> LGTM, of course (more tests, yay!).
>
> The indentation makes this hard to read though. Maybe declare the lambda as a separate variable?

The indentation is clang-format's doing. I'm a bit hesitant to put the lambda in a variable because it's only used in one place so putting it inline is the more standard thing to do.
However, I don't have a super strong opinion here so if you think it's clearer then I'll make the change.

Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D54861