search

marceldev89 / BattleNET

BattlEye Protocol Library and Client
GNU Lesser General Public License v3.0
76 stars 44 forks source link

100 attempts to reconnect #46

Closed tym32167 closed 6 years ago

tym32167 commented 6 years ago

regarding #34 Had a dosen issues because of this. So, in my code I decided to make 100 attempts to connect, if all 100 attempts will fail - so, means that connect not possible. Testing this for a couple of days, me (3 instances of app) + ~5 additional users. Worked fine, at least, not crashed. Stackoverflow very annoying exception since it is not catchable and it destroy application each time.

tym32167 commented 6 years ago

@marceldev89 sir yes sir :) Done by new commit.

marceldev89 commented 6 years ago

Oops, forgot 1 small thingy. The ConnectInternal method should probably be private.. sorry :)

tym32167 commented 6 years ago

@marceldev89 no worries, my bad. I wanted it private as well. Done

marceldev89 commented 6 years ago

Thank you. Merged. :smiley: