krageon/openmw-tes3coop
1
0
Fork 0
forked from teamnwah/openmw-tes3coop
Code Issues Pull requests Releases Wiki Activity

[General] Add comments to false-positive PVS messages

Browse source
This commit is contained in:
Koncord 2019-01-04 13:03:31 +08:00
parent ddd370b61b
commit 6caed5d47e
10 changed files with 13 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
apps/openmw-mp/processors/ObjectProcessor.cpp
View file

@ -31,7 +31,7 @@ bool ObjectProcessor::Process(RakNet::Packet &packet, BaseObjectList &objectList
if (!processor.second->avoidReading)
myPacket->Read();
if (objectList.isValid) // -V547
if (objectList.isValid) // -V547 (PVS Studio false-positive warning)
processor.second->Do(*myPacket, *player, objectList);
else
LOG_MESSAGE_SIMPLE(Log::LOG_ERROR, "Received %s that failed integrity check and was ignored!", processor.second->strPacketID.c_str());

2
apps/openmw-mp/processors/WorldstateProcessor.cpp
View file

@ -28,7 +28,7 @@ bool WorldstateProcessor::Process(RakNet::Packet &packet, BaseWorldstate &worlds
if (!processor.second->avoidReading)
myPacket->Read();
if (worldstate.isValid) // -V547
if (worldstate.isValid) // -V547 (PVS Studio false-positive warning)
processor.second->Do(*myPacket, *player, worldstate);
else
LOG_MESSAGE_SIMPLE(Log::LOG_ERROR, "Received %s that failed integrity check and was ignored!", processor.second->strPacketID.c_str());

13
apps/openmw/mwmp/MechanicsHelper.cpp
View file

@ -163,14 +163,11 @@ mwmp::Target MechanicsHelper::getTarget(const MWWorld::Ptr& ptr)
{
MWWorld::CellRef *ptrRef = &ptr.getCellRef();
if (ptrRef)
{
target.isPlayer = false;
target.refId = ptrRef->getRefId();
target.refNum = ptrRef->getRefNum().mIndex;
target.mpNum = ptrRef->getMpNum();
target.name = ptr.getClass().getName(ptr);
}
target.isPlayer = false;
target.refId = ptrRef->getRefId();
target.refNum = ptrRef->getRefNum().mIndex;
target.mpNum = ptrRef->getMpNum();
target.name = ptr.getClass().getName(ptr);
}
}

2
apps/openmw/mwmp/RecordHelper.cpp
View file

@ -474,8 +474,6 @@ void RecordHelper::overrideArmorRecord(const mwmp::ArmorRecord& record)
ESM::Armor finalData = *baseData;
finalData.mId = recordData.mId;
finalData.mParts.mParts.at(0);
if (record.baseOverrides.hasName)
finalData.mName = recordData.mName;

2
apps/openmw/mwmp/processors/ActorProcessor.cpp
View file

@ -36,7 +36,7 @@ bool ActorProcessor::Process(RakNet::Packet &packet, ActorList &actorList)
myPacket->Read();
}
if (actorList.isValid)
if (actorList.isValid) // -V547 (PVS Studio false-positive warning)
processor.second->Do(*myPacket, actorList);
else
LOG_MESSAGE_SIMPLE(Log::LOG_ERROR, "Received %s that failed integrity check and was ignored!", processor.second->strPacketID.c_str());

2
apps/openmw/mwmp/processors/ObjectProcessor.cpp
View file

@ -31,7 +31,7 @@ bool ObjectProcessor::Process(RakNet::Packet &packet, ObjectList &objectList)
if (!request && !processor.second->avoidReading)
myPacket->Read();
if (objectList.isValid)
if (objectList.isValid) // -V547 (PVS Studio false-positive warning)
processor.second->Do(*myPacket, objectList);
else
LOG_MESSAGE_SIMPLE(Log::LOG_ERROR, "Received %s that failed integrity check and was ignored!", processor.second->strPacketID.c_str());

2
apps/openmw/mwmp/processors/WorldstateProcessor.cpp
View file

@ -31,7 +31,7 @@ bool WorldstateProcessor::Process(RakNet::Packet &packet, Worldstate &worldstate
if (!request && !processor.second->avoidReading)
myPacket->Read();
if (worldstate.isValid)
if (worldstate.isValid) // -V547 (PVS Studio false-positive warning)
processor.second->Do(*myPacket, worldstate);
else
LOG_MESSAGE_SIMPLE(Log::LOG_ERROR, "Received %s that failed integrity check and was ignored!", processor.second->strPacketID.c_str());

2
apps/openmw/mwmp/processors/player/ProcessorPlayerCellState.hpp
View file

@ -11,7 +11,7 @@ namespace mwmp
ProcessorPlayerCellState()
{
BPP_INIT(ID_PLAYER_CELL_STATE)
avoidReading = true;
avoidReading = true; // -V519 (PVS Studio false-positive warning)
}
virtual void Do(PlayerPacket &packet, BasePlayer *player)

2
apps/openmw/mwmp/processors/player/ProcessorPlayerDeath.hpp
View file

@ -16,7 +16,7 @@ namespace mwmp
ProcessorPlayerDeath()
{
BPP_INIT(ID_PLAYER_DEATH)
avoidReading = true;
avoidReading = true; // -V519 (PVS Studio false-positive warning)
}
virtual void Do(PlayerPacket &packet, BasePlayer *player)

2
apps/openmw/mwmp/processors/player/ProcessorUserDisconnected.hpp
View file

@ -18,7 +18,7 @@ namespace mwmp
ProcessorUserDisconnected()
{
BPP_INIT(ID_USER_DISCONNECTED)
avoidReading = true;
avoidReading = true; // -V519 (PVS Studio false-positive warning)
}
virtual void Do(PlayerPacket &packet, BasePlayer *player)