From 90be8fafb39469bd2c318c033e63294ebaad2ca4 Mon Sep 17 00:00:00 2001 From: Shauren Date: Sun, 30 Nov 2025 14:25:32 +0100 Subject: Core/Misc: Use std::scoped_lock instead of unique_lock where possible (and old lock_guard) --- src/server/shared/Realm/RealmList.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/server/shared/Realm') diff --git a/src/server/shared/Realm/RealmList.cpp b/src/server/shared/Realm/RealmList.cpp index c8b4926ced6..56bef36217a 100644 --- a/src/server/shared/Realm/RealmList.cpp +++ b/src/server/shared/Realm/RealmList.cpp @@ -187,7 +187,7 @@ void RealmList::UpdateRealms() TC_LOG_INFO("realmlist", "Removed realm \"{}\".", itr->second); { - std::unique_lock lock(_realmsMutex); + std::scoped_lock lock(_realmsMutex); _subRegions.swap(newSubRegions); _realms.swap(newRealms); @@ -213,7 +213,7 @@ void RealmList::UpdateRealms() std::shared_ptr RealmList::GetRealm(Battlenet::RealmHandle const& id) const { - std::shared_lock lock(_realmsMutex); + std::shared_lock lock(_realmsMutex); return Trinity::Containers::MapGetValuePtr(_realms, id); } @@ -236,7 +236,7 @@ std::shared_ptr RealmList::GetCurrentRealm() const void RealmList::WriteSubRegions(bgs::protocol::game_utilities::v1::GetAllValuesForAttributeResponse* response) const { - std::shared_lock lock(_realmsMutex); + std::shared_lock lock(_realmsMutex); for (std::string const& subRegion : _subRegions) response->add_attribute_value()->set_string_value(subRegion); } @@ -301,7 +301,7 @@ std::vector RealmList::GetRealmList(uint32 build, AccountTypes accountSec { JSON::RealmList::RealmListUpdates realmList; { - std::shared_lock lock(_realmsMutex); + std::shared_lock lock(_realmsMutex); for (auto const& [_, realm] : _realms) { if (realm->Id.GetSubRegionAddress() != subRegion) -- cgit v1.2.3