Split this for good practice, making it a bit easier to read and maintain.
- rP22916: Split NetFileTransfer message parsing into one function per case, refs #5212.
- Trac Tickets
Notice the code didn't change. Could use an FSM model eventually.
|106 ↗||(On Diff #6738)|
I guess it's exactly 0.5 because that's 1/2 :p as in at most two LOG draws per second.
The sprintf casts can be removed by using %lu for the u32 arguments and %zu for size_t. The latter has a precedent in the VertexBufferManager.cpp:
size_t vertexSize, size_t numVertices
LOGERROR("Failed to create VBOs (%zu*%zu)", vertexSize, numVertices);
|123 ↗||(On Diff #9815)|
We can reuse the iterator below to avoid the duplicate lookup. Also the  operator has the side effect of insertion when most of the time it would be an error to insert this way.
|91 ↗||(On Diff #9815)|