From 7c7b4760d89f29b7ee3c2f15f94cd9be24dc36cb Mon Sep 17 00:00:00 2001 From: Shauren Date: Mon, 16 Aug 2021 11:29:54 +0200 Subject: Scripts/Commands: Avoid copies of RBACCommandData Silences the PVS V1002 warning: "The 'RBACCommandData' class, containing pointers, constructor and destructor, is copied by the automatically generated copy constructor." --- src/server/scripts/Commands/cs_rbac.cpp | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) (limited to 'src') diff --git a/src/server/scripts/Commands/cs_rbac.cpp b/src/server/scripts/Commands/cs_rbac.cpp index f2a2ce1e42c..622b625ee3c 100644 --- a/src/server/scripts/Commands/cs_rbac.cpp +++ b/src/server/scripts/Commands/cs_rbac.cpp @@ -34,15 +34,15 @@ EndScriptData */ struct RBACCommandData { - RBACCommandData() : rbac(nullptr), needDelete(false) {} + RBACCommandData(RBACCommandData const&) = delete; ~RBACCommandData() { if (needDelete) delete rbac; } - rbac::RBACData* rbac; - bool needDelete; + rbac::RBACData* rbac = nullptr; + bool needDelete = false; }; using namespace Trinity::ChatCommands; @@ -78,18 +78,13 @@ public: static RBACCommandData GetRBACData(AccountIdentifier account) { - RBACCommandData data = RBACCommandData(); - if (account.IsConnected()) - data.rbac = account.GetConnectedSession()->GetRBACData(); - else - { - data.rbac = new rbac::RBACData(account.GetID(), account.GetName(), realm.Id.Realm, AccountMgr::GetSecurity(account.GetID(), realm.Id.Realm)); - data.rbac->LoadFromDB(); - data.needDelete = true; - } + return { account.GetConnectedSession()->GetRBACData(), false }; + + rbac::RBACData* rbac = new rbac::RBACData(account.GetID(), account.GetName(), realm.Id.Realm, AccountMgr::GetSecurity(account.GetID(), realm.Id.Realm)); + rbac->LoadFromDB(); - return data; + return { rbac, true }; } static bool HandleRBACPermGrantCommand(ChatHandler* handler, Optional account, uint32 permId, Optional realmId) -- cgit v1.2.3