From 544de86739a85792be7e6883968b2af8590d0711 Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Sun, 31 Aug 2014 14:11:23 +0300
Subject: Add comments for outbound packet fields in debug log.

---
 src/net/tmwa/messageout.cpp | 17 +++++++++--------
 src/net/tmwa/messageout.h   | 11 ++++++++---
 2 files changed, 17 insertions(+), 11 deletions(-)

(limited to 'src/net/tmwa')

diff --git a/src/net/tmwa/messageout.cpp b/src/net/tmwa/messageout.cpp
index 31196a4a7..a740df563 100644
--- a/src/net/tmwa/messageout.cpp
+++ b/src/net/tmwa/messageout.cpp
@@ -46,7 +46,7 @@ MessageOut::MessageOut(const int16_t id) :
     mData = mNetwork->mOutBuffer + static_cast<size_t>(mNetwork->mOutSize);
 
     // +++ can be issue. call to virtual member
-    writeInt16(id);
+    writeInt16(id, "packet id");
 }
 
 void MessageOut::expand(const size_t bytes)
@@ -55,9 +55,9 @@ void MessageOut::expand(const size_t bytes)
     PacketCounters::incOutBytes(static_cast<int>(bytes));
 }
 
-void MessageOut::writeInt16(const int16_t value)
+void MessageOut::writeInt16(const int16_t value, const char *const str)
 {
-    DEBUGLOG("writeInt16: " + toStringPrint(static_cast<int>(value)));
+    DEBUGLOG2("writeInt16: " + toStringPrint(static_cast<int>(value)), str);
     expand(2);
 #if SDL_BYTEORDER == SDL_BIG_ENDIAN
     int16_t swap = SDL_Swap16(value);
@@ -69,9 +69,9 @@ void MessageOut::writeInt16(const int16_t value)
     PacketCounters::incOutBytes(2);
 }
 
-void MessageOut::writeInt32(const int32_t value)
+void MessageOut::writeInt32(const int32_t value, const char *const str)
 {
-    DEBUGLOG("writeInt32: " + toStringPrint(value));
+    DEBUGLOG2("writeInt32: " + toStringPrint(value), str);
     expand(4);
 #if SDL_BYTEORDER == SDL_BIG_ENDIAN
     int32_t swap = SDL_Swap32(value);
@@ -89,11 +89,12 @@ static_cast<uint16_t>(w)) >> 8U))
 
 void MessageOut::writeCoordinates(const uint16_t x,
                                   const uint16_t y,
-                                  unsigned char direction)
+                                  unsigned char direction,
+                                  const char *const str)
 {
-    DEBUGLOG(strprintf("writeCoordinates: %u,%u %u",
+    DEBUGLOG2(strprintf("writeCoordinates: %u,%u %u",
         static_cast<unsigned>(x), static_cast<unsigned>(y),
-        static_cast<unsigned>(direction)));
+        static_cast<unsigned>(direction)), str);
     unsigned char *const data = reinterpret_cast<unsigned char*>(mData)
         + static_cast<size_t>(mPos);
     mNetwork->mOutSize += 3;
diff --git a/src/net/tmwa/messageout.h b/src/net/tmwa/messageout.h
index 57ba9e75b..7aba1677b 100644
--- a/src/net/tmwa/messageout.h
+++ b/src/net/tmwa/messageout.h
@@ -47,16 +47,21 @@ class MessageOut final : public Net::MessageOut
 
         A_DELETE_COPY(MessageOut)
 
-        void writeInt16(const int16_t value);        /**< Writes a short. */
+        /**< Writes a short. */
+        void writeInt16(const int16_t value,
+                        const char *const str = nullptr);
 
-        void writeInt32(const int32_t value);        /**< Writes a long. */
+        /**< Writes a long. */
+        void writeInt32(const int32_t value,
+                        const char *const str = nullptr);
 
         /**
          * Encodes coordinates and direction in 3 bytes.
          */
         void writeCoordinates(const uint16_t x,
                               const uint16_t y,
-                              unsigned char direction);
+                              unsigned char direction,
+                              const char *const str = nullptr);
 
         void resetPos()
         { mPos = 0; }
-- 
cgit v1.2.3-70-g09d2