PointLockManager::UnLockKey: use swap instead of check

topling · Jun 25, 2022 · 9d8106d · 9d8106d
1 parent 9f2091f
commit 9d8106d
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/utilities/transactions/lock/point/point_lock_manager.cc b/utilities/transactions/lock/point/point_lock_manager.cc
@@ -587,10 +587,7 @@ void PointLockManager::UnLockKey(PessimisticTransaction* txn,
       if (txns.size() == 1) {
         stripe->keys.erase(stripe_iter);
       } else {
-        auto last_it = txns.end() - 1;
-        if (txn_it != last_it) {
-          *txn_it = *last_it;
-        }
+        std::swap(txns.back(), *txn_it);
         txns.pop_back();
       }