Page MenuHomeWildfire Games

Always try to flush netmessages when closing the NetClient
Needs ReviewPublic

Authored by elexis on Jun 5 2018, 4:59 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

rP14732 implemented flushing code when destroying the NetClient that actually succeeds in 1 out of 10 cases, refs D1557.
If we don't remove this code entirely (might have the danger of people unconsciously reintroducing this), then I don't see why we don't always flush the messages.

Test Plan

Make sure that this doesn't break anything. Not sure how, test alt+f4 at all stages of the client.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6223
Build 10336: Vulcan BuildJenkins
Build 10335: arc lint + arc unit