diff options
| author | Amin Hassani <ahassani@google.com> | 2017-12-06 13:47:52 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-12-12 13:50:58 -0800 |
| commit | d8b67f49d7f8ae291cd4fa901b2d3767137704e7 (patch) | |
| tree | 2d039015afcd03433da0dd8860a83ce3ce2a65cf /payload_generator/inplace_generator.cc | |
| parent | 06c6d088b89656a120d40c4b578200c87af5c4c8 (diff) | |
update_engine: Remove the duplicate BlocksInExtents
This patch removes the duplicate BlocksInExtents from extent_utils.h and fixes
the remainder of the code to reflect this change.
BUG=none
TEST=unittests pass;
Change-Id: I76f5106f75072b20cd8f41f081b2f2b07aeac9a8
Reviewed-on: https://chromium-review.googlesource.com/812009
Commit-Ready: Amin Hassani <ahassani@chromium.org>
Tested-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Alex Deymo <deymo@google.com>
Reviewed-by: Sen Jiang <senj@chromium.org>
Diffstat (limited to 'payload_generator/inplace_generator.cc')
| -rw-r--r-- | payload_generator/inplace_generator.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/payload_generator/inplace_generator.cc b/payload_generator/inplace_generator.cc index cf02c913..b858c2be 100644 --- a/payload_generator/inplace_generator.cc +++ b/payload_generator/inplace_generator.cc @@ -299,8 +299,7 @@ namespace { template<typename T> bool TempBlocksExistInExtents(const T& extents) { - for (int i = 0, e = extents.size(); i < e; ++i) { - Extent extent = GetElement(extents, i); + for (const auto& extent : extents) { uint64_t start = extent.start_block(); uint64_t num = extent.num_blocks(); if (start >= kTempBlockStart || (start + num) >= kTempBlockStart) { |