Let `erase()` return new iterator instead of doing it manually

1 files changed, 1 insertions, 1 deletions

diff --git a/messagebus/src/vespa/messagebus/network/rpctargetpool.cpp b/messagebus/src/vespa/messagebus/network/rpctargetpool.cpp
index 1c7dd4dd453..b403c65f863 100644
--- a/messagebus/src/vespa/messagebus/network/rpctargetpool.cpp
+++ b/messagebus/src/vespa/messagebus/network/rpctargetpool.cpp
@@ -65,7 +65,7 @@ RPCTargetPool::flushTargets(bool force)
             const Entry& entry = it->second;
             if (!entry.inUse(guard) && (force || ((entry.lastUse() + _expireMillis) < currentTime))) {
                 to_erase_on_scope_exit.emplace_back(std::move(it->second));
-                _targets.erase(it++); // postfix increment to move the iterator
+                it = _targets.erase(it);
             } else {
                 ++it;
             }