diff options
author | jackpoz <giacomopoz@gmail.com> | 2017-04-25 17:33:53 +0200 |
---|---|---|
committer | jackpoz <giacomopoz@gmail.com> | 2017-04-25 17:33:53 +0200 |
commit | 486b03234304526d8f7d66ff0d99ea3de975745e (patch) | |
tree | 8e66523c1688a27d88a6ae64860378fe0ecb9ede /src | |
parent | 3194d089cbf2dd30bb3fed3a7f6f7e85730ad0cb (diff) |
Core/Misc: Fix static analysis issues
Diffstat (limited to 'src')
-rw-r--r-- | src/common/Collision/Maps/TileAssembler.cpp | 9 | ||||
-rw-r--r-- | src/server/game/Entities/Player/Player.cpp | 2 |
2 files changed, 5 insertions, 6 deletions
diff --git a/src/common/Collision/Maps/TileAssembler.cpp b/src/common/Collision/Maps/TileAssembler.cpp index cbd468d3337..2c4d3105050 100644 --- a/src/common/Collision/Maps/TileAssembler.cpp +++ b/src/common/Collision/Maps/TileAssembler.cpp @@ -216,18 +216,17 @@ namespace VMAP return false; } printf("Read coordinate mapping...\n"); - uint32 mapID, tileX, tileY, check=0; + uint32 mapID, tileX, tileY, check; G3D::Vector3 v1, v2; ModelSpawn spawn; while (!feof(dirf)) { - check = 0; // read mapID, tileX, tileY, Flags, adtID, ID, Pos, Rot, Scale, Bound_lo, Bound_hi, name - check += fread(&mapID, sizeof(uint32), 1, dirf); + check = fread(&mapID, sizeof(uint32), 1, dirf); if (check == 0) // EoF... break; - check += fread(&tileX, sizeof(uint32), 1, dirf); - check += fread(&tileY, sizeof(uint32), 1, dirf); + fread(&tileX, sizeof(uint32), 1, dirf); + fread(&tileY, sizeof(uint32), 1, dirf); if (!ModelSpawn::readFromFile(dirf, spawn)) break; diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index e359ec8af4f..d2420ca9367 100644 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -12507,7 +12507,7 @@ void Player::DestroyItem(uint8 bag, uint8 slot, bool update) { // item set bonuses applied only at equip and removed at unequip, and still active for broken items - if (pProto && pProto->ItemSet) + if (pProto->ItemSet) RemoveItemsSetItem(this, pProto); _ApplyItemMods(pItem, slot, false); |