View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
0001439 | OpenClonk | Engine - Network | public | 2015-11-19 06:29 | 2017-12-26 18:42 | ||||||||
Reporter | Sven2 | ||||||||||||
Assigned To | |||||||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||||||
Status | new | Resolution | open | ||||||||||
Platform | Visual C++ 2008 | OS | Windows | OS Version | |||||||||
Product Version | |||||||||||||
Target Version | git master | Fixed in Version | |||||||||||
Summary | 0001439: Ready button race condition | ||||||||||||
Description | The new ready button has a minor race condition: If a client joins while another client is pressing the ready button, the new client might not get the correct ready-state. The simple solution is to let clients send the ready-packet to the host and let the host re-distribute it to all clients. Also, force update of client list on ready packet in lobby. And maybe change the read-icon to a computer plus checkmark instead of player portrait plus checkmark. It looks a bit odd when the icon suddenly changes from a computer to a portrait. (Although it's not completely wrong if you consider non-ready as "afk" and ready as "player is here") It's minor and otherwise the ready button is good for now, so I'll put this on low priority (i.e. not release-blocking). | ||||||||||||
Tags | No tags attached. | ||||||||||||
Attached Files |
|