summaryrefslogtreecommitdiff
path: root/src/net/messagein.cpp
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2014-08-31 13:53:06 +0300
committerAndrei Karas <akaras@inbox.ru>2014-08-31 13:53:06 +0300
commita63248380554067d4291c653b4ba82db36fbc821 (patch)
treec4ad257d15fc44179ccafe9c3e887bfce710967c /src/net/messagein.cpp
parent1d5aa3c38c3a545ad0056613f561ac01d945b2ed (diff)
downloadmv-a63248380554067d4291c653b4ba82db36fbc821.tar.gz
mv-a63248380554067d4291c653b4ba82db36fbc821.tar.bz2
mv-a63248380554067d4291c653b4ba82db36fbc821.tar.xz
mv-a63248380554067d4291c653b4ba82db36fbc821.zip
Add comments for inbound packet fields in debug log.
Diffstat (limited to 'src/net/messagein.cpp')
-rw-r--r--src/net/messagein.cpp47
1 files changed, 26 insertions, 21 deletions
diff --git a/src/net/messagein.cpp b/src/net/messagein.cpp
index ccc80a838..b131e85e0 100644
--- a/src/net/messagein.cpp
+++ b/src/net/messagein.cpp
@@ -47,7 +47,7 @@ MessageIn::MessageIn(const char *const data, const unsigned int length) :
DEBUGLOG("MessageIn");
}
-unsigned char MessageIn::readUInt8()
+unsigned char MessageIn::readUInt8(const char *const str)
{
unsigned char value = static_cast<unsigned char>(-1);
if (mPos < mLength)
@@ -55,11 +55,11 @@ unsigned char MessageIn::readUInt8()
mPos += 1;
PacketCounters::incInBytes(1);
- DEBUGLOG("readUInt8: " + toStringPrint(static_cast<int>(value)));
+ DEBUGLOG2("readUInt8: " + toStringPrint(static_cast<int>(value)), str);
return value;
}
-signed char MessageIn::readInt8()
+signed char MessageIn::readInt8(const char *const str)
{
signed char value = static_cast<signed char>(-1);
if (mPos < mLength)
@@ -67,7 +67,7 @@ signed char MessageIn::readInt8()
mPos += 1;
PacketCounters::incInBytes(1);
- DEBUGLOG("readInt8: " + toStringPrint(static_cast<int>(value)));
+ DEBUGLOG2("readInt8: " + toStringPrint(static_cast<int>(value)), str);
return value;
}
@@ -101,8 +101,10 @@ uint8_t MessageIn::fromServerDirection(const uint8_t serverDir)
}
}
-void MessageIn::readCoordinates(uint16_t &restrict x, uint16_t &restrict y,
- uint8_t &restrict direction)
+void MessageIn::readCoordinates(uint16_t &restrict x,
+ uint16_t &restrict y,
+ uint8_t &restrict direction,
+ const char *const str)
{
if (mPos + 3 <= mLength)
{
@@ -115,10 +117,10 @@ void MessageIn::readCoordinates(uint16_t &restrict x, uint16_t &restrict y,
const uint8_t serverDir = static_cast<uint8_t>(data[2] & 0x000f);
direction = fromServerDirection(serverDir);
- DEBUGLOG(std::string("readCoordinates: ").append(toString(
+ DEBUGLOG2(std::string("readCoordinates: ").append(toString(
static_cast<int>(x))).append(",").append(toString(
static_cast<int>(y))).append(",").append(toString(
- static_cast<int>(serverDir))));
+ static_cast<int>(serverDir))), str);
}
else
{
@@ -134,7 +136,8 @@ void MessageIn::readCoordinates(uint16_t &restrict x, uint16_t &restrict y,
void MessageIn::readCoordinatePair(uint16_t &restrict srcX,
uint16_t &restrict srcY,
uint16_t &restrict dstX,
- uint16_t &restrict dstY)
+ uint16_t &restrict dstY,
+ const char *const str)
{
if (mPos + 5 <= mLength)
{
@@ -150,11 +153,11 @@ void MessageIn::readCoordinatePair(uint16_t &restrict srcX,
temp = MAKEWORD(data[2], data[1] & 0x003f);
srcY = static_cast<uint16_t>(temp >> 4);
- DEBUGLOG(std::string("readCoordinatePair: ").append(toString(
+ DEBUGLOG2(std::string("readCoordinatePair: ").append(toString(
static_cast<int>(srcX))).append(",").append(toString(
static_cast<int>(srcY))).append(" ").append(toString(
static_cast<int>(dstX))).append(",").append(toString(
- static_cast<int>(dstY))));
+ static_cast<int>(dstY))), str);
}
else
{
@@ -168,14 +171,14 @@ void MessageIn::readCoordinatePair(uint16_t &restrict srcX,
PacketCounters::incInBytes(5);
}
-void MessageIn::skip(const unsigned int length)
+void MessageIn::skip(const unsigned int length, const char *const str)
{
mPos += length;
PacketCounters::incInBytes(length);
- DEBUGLOG("skip: " + toString(static_cast<int>(length)));
+ DEBUGLOG2("skip: " + toString(static_cast<int>(length)), str);
}
-std::string MessageIn::readString(int length)
+std::string MessageIn::readString(int length, const char *const dstr)
{
// Get string length
if (length < 0)
@@ -185,7 +188,7 @@ std::string MessageIn::readString(int length)
if (length < 0 || mPos + length > mLength)
{
mPos = mLength + 1;
- DEBUGLOG("readString error");
+ DEBUGLOG2("readString error", dstr);
return "";
}
@@ -198,11 +201,11 @@ std::string MessageIn::readString(int length)
? stringEnd - stringBeg : static_cast<size_t>(length));
mPos += length;
PacketCounters::incInBytes(length);
- DEBUGLOG("readString: " + str);
+ DEBUGLOG2("readString: " + str, dstr);
return str;
}
-std::string MessageIn::readRawString(int length)
+std::string MessageIn::readRawString(int length, const char *const dstr)
{
// Get string length
if (length < 0)
@@ -224,7 +227,7 @@ std::string MessageIn::readRawString(int length)
mPos += length;
PacketCounters::incInBytes(length);
- DEBUGLOG("readString: " + str);
+ DEBUGLOG2("readString: " + str, dstr);
if (stringEnd)
{
@@ -237,7 +240,7 @@ std::string MessageIn::readRawString(int length)
stringEnd2 ? stringEnd2 - stringBeg2 : len2);
if (hiddenPart.length() > 0)
{
- DEBUGLOG("readString2: " + hiddenPart);
+ DEBUGLOG2("readString2: " + hiddenPart, dstr);
return str.append("|").append(hiddenPart);
}
}
@@ -245,7 +248,7 @@ std::string MessageIn::readRawString(int length)
return str;
}
-unsigned char *MessageIn::readBytes(int length)
+unsigned char *MessageIn::readBytes(int length, const char *const dstr)
{
// Get string length
if (length < 0)
@@ -255,7 +258,7 @@ unsigned char *MessageIn::readBytes(int length)
if (length < 0 || mPos + length > mLength)
{
mPos = mLength + 1;
- DEBUGLOG("readBytesString error");
+ DEBUGLOG2("readBytesString error", dstr);
return nullptr;
}
@@ -279,6 +282,8 @@ unsigned char *MessageIn::readBytes(int length)
else
str.append("_");
}
+ if (dstr)
+ logger->dlog(dstr);
logger->dlog("ReadBytes: " + str);
#endif