search

lidgren / lidgren-network-gen3

Lidgren Network Library
https://groups.google.com/forum/#!forum/lidgren-network-gen3
MIT License
1.19k stars 331 forks source link

Fix: Connection.Connect() now correctly updates the Connection.Status… #118

Closed nekoniaow closed 5 years ago

nekoniaow commented 5 years ago

Fix: Connection.Connect() now correctly updates the Connection.Status property when needed.

Previously, it would never update it, even when users had opted to not receive status update change messages. It now properly calls SetStatus() instead of modifying the m_status field directly.

This fixes issue #117.

nekoniaow commented 5 years ago

@lidgren , hi, sorry to ping you out of nowhere but do you think this fix has any chances to be incorporated? (linked issue contains additional materials, including sample code which demonstrates the issue). Thanks! ;)

lidgren commented 5 years ago

Thanks for the fix!