From e6f26cfb1f75d21a166cc3fd371f9c5f1641625e Mon Sep 17 00:00:00 2001 From: jackpoz Date: Sun, 8 Feb 2015 19:40:03 +0100 Subject: Core/Misc: Fix static analysis issues Fix some static analysis issues reported by Coverity. Rename SpellMgr::EnsureSpellInfo() to SpellMgr::AssertSpellInfo() to be consistent with other similar methods. --- src/server/game/Pools/PoolMgr.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/server/game/Pools/PoolMgr.cpp') diff --git a/src/server/game/Pools/PoolMgr.cpp b/src/server/game/Pools/PoolMgr.cpp index 257c279ade2..9f8a9f8686b 100644 --- a/src/server/game/Pools/PoolMgr.cpp +++ b/src/server/game/Pools/PoolMgr.cpp @@ -170,7 +170,7 @@ PoolObject* PoolGroup::RollOne(ActivePoolData& spawns, uint64 triggerFrom) } if (!EqualChanced.empty()) { - int32 index = irand(0, EqualChanced.size()-1); + uint32 index = urand(0, EqualChanced.size()-1); // Triggering object is marked as spawned at this time and can be also rolled (respawn case) // so this need explicit check for this case if (EqualChanced[index].guid == triggerFrom || !spawns.IsActiveObject(EqualChanced[index].guid)) -- cgit v1.2.3