Stop NetworkPacket methods from producing bloated packets

This commit is contained in:
Jay Arndt 2015-05-04 04:53:41 -05:00
parent 5fa4fed097
commit 22e1d26264
2 changed files with 5 additions and 17 deletions

View File

@ -72,8 +72,8 @@ char* NetworkPacket::getString(u32 from_offset)
void NetworkPacket::putRawString(const char* src, u32 len)
{
if (m_read_offset + len * sizeof(char) >= m_datasize) {
m_datasize += len * sizeof(char);
if (m_read_offset + len * sizeof(char) > m_datasize) {
m_datasize = m_read_offset + len * sizeof(char);
m_data.resize(m_datasize);
}
@ -108,13 +108,7 @@ NetworkPacket& NetworkPacket::operator<<(std::string src)
*this << msgsize;
if (m_read_offset + msgsize * sizeof(char) >= m_datasize) {
m_datasize += msgsize * sizeof(char);
m_data.resize(m_datasize);
}
memcpy(&m_data[m_read_offset], src.c_str(), msgsize);
m_read_offset += msgsize;
putRawString(src.c_str(), (u32)msgsize);
return *this;
}
@ -128,13 +122,7 @@ void NetworkPacket::putLongString(std::string src)
*this << msgsize;
if (m_read_offset + msgsize * sizeof(char) >= m_datasize) {
m_datasize += msgsize * sizeof(char);
m_data.resize(m_datasize);
}
memcpy(&m_data[m_read_offset], src.c_str(), msgsize);
m_read_offset += msgsize;
putRawString(src.c_str(), msgsize);
}
NetworkPacket& NetworkPacket::operator>>(std::wstring& dst)

View File

@ -114,7 +114,7 @@ private:
template<typename T> void checkDataSize()
{
if (m_read_offset + sizeof(T) > m_datasize) {
m_datasize += sizeof(T);
m_datasize = m_read_offset + sizeof(T);
m_data.resize(m_datasize);
}
}