From a65224d39e442e11d45758e155907f9ddb9a3bd1 Mon Sep 17 00:00:00 2001
From: EntireTwix <willkatz.dev@gmail.com>
Date: Tue, 13 Jul 2021 16:31:39 -0700
Subject: [PATCH] :racehorse::truck: moved shared lock to decrease scope
 accurately

---
 src/bank.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bank.cpp b/src/bank.cpp
index 983a3da..0e8300d 100644
--- a/src/bank.cpp
+++ b/src/bank.cpp
@@ -245,7 +245,6 @@ BankResponse Bank::AddUser(const std::string &name, uint32_t init_bal, std::stri
 }
 BankResponse Bank::DelUser(const std::string &name) noexcept
 {
-    std::shared_lock<std::shared_mutex> lock{save_lock};
 #if RETURN_ON_DEL
     uint32_t bal;
     if (users.if_contains(name, [&bal](const User &u) { bal = u.balance; }) && bal)
@@ -256,6 +255,7 @@ BankResponse Bank::DelUser(const std::string &name) noexcept
         }
     }
 #endif
+    std::shared_lock<std::shared_mutex> lock{save_lock};
     if (users.erase(name))
     {
 #if CONSERVATIVE_DISK_SAVE