Reserve 4k of buffer for CSerializedNetMsg buffer

codablock · panleone · commit 04b6a67c413b · 2024-04-07T13:02:22.000+02:00
Profiling has shown that a lot of time is spent in resizing the data
vector when large messages are involved.
diff --git a/src/netmessagemaker.h b/src/netmessagemaker.h
@@ -18,6 +18,7 @@ class CNetMsgMaker
     {
         CSerializedNetMsg msg;
         msg.command = std::move(sCommand);
+        msg.data.reserve(4 * 1024);
         CVectorWriter{ SER_NETWORK, nFlags | nVersion, msg.data, 0, std::forward<Args>(args)... };
         return msg;
     }

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@ class CNetMsgMaker`
`18`	`18`	`{`
`19`	`19`	`CSerializedNetMsg msg;`
`20`	`20`	`msg.command = std::move(sCommand);`
	`21`	`+ msg.data.reserve(4 * 1024);`
`21`	`22`	`CVectorWriter{ SER_NETWORK, nFlags \| nVersion, msg.data, 0, std::forward<Args>(args)... };`
`22`	`23`	`return msg;`
`23`	`24`	`}`