It makes sense and there also was a TODO about that.
Details
- Reviewers
elexis - Commits
- rP22012: [NetServer] Split `OnInGame` function
Verify the behaviour is the same as before and all relevant code was copied. Agree on the new function names
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 6125 Build 10196: Vulcan Build Jenkins Build 10195: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/547/display/redirect
source/network/NetServer.cpp | ||
---|---|---|
1110 | static_cast here and below. |
source/network/NetServer.cpp | ||
---|---|---|
1110 | If we decide to do that, we should do it consistently in all the network files. |
Yes, this split is overdue! There is nothing else to split, all lines that I read are copy&pasted correctly, I hope the other ones are copy&pasted correctly too, variable and function names are ideal.
source/network/NetServer.cpp | ||
---|---|---|
1126 | (I was wondering if we shouldn't cache that, but probably not really changing the performance as a simulation command takes so much more performance and we may want to support changing some settings dynamically eventually) |