From d6b086ed03d2984186e7d448a3ae4b81a1f61c4e Mon Sep 17 00:00:00 2001 From: Marcus Holland-Moritz Date: Tue, 15 Dec 2020 14:17:47 +0100 Subject: [PATCH] Delay definition of `finalize_inode` (This doesn't make any difference in behaviour.) --- src/dwarfs/inode_manager.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/dwarfs/inode_manager.cpp b/src/dwarfs/inode_manager.cpp index 98c04e4b..cccf1a29 100644 --- a/src/dwarfs/inode_manager.cpp +++ b/src/dwarfs/inode_manager.cpp @@ -333,6 +333,9 @@ void inode_manager_::order_inodes_by_nilsimsa( index.resize(count); std::iota(index.begin(), index.end(), 0); + auto empty = std::partition(index.begin(), index.end(), + [&](auto i) { return inodes[i]->size() > 0; }); + auto finalize_inode = [&]() { inodes_.push_back(std::move(inodes[index.back()])); index.pop_back(); @@ -340,9 +343,6 @@ void inode_manager_::order_inodes_by_nilsimsa( fn(inodes_.back()); }; - auto empty = std::partition(index.begin(), index.end(), - [&](auto i) { return inodes[i]->size() > 0; }); - if (empty != index.end()) { assert(empty + 1 == index.end()); finalize_inode();