Testing Bug 759

[ILMTitan]

I have a thought that might fix bug 759, but I have no idea how to test it. Any ideas? Also could I get a sanity check from someone more familiar with that section of the code (CNet & CNetProtocol)?

[Auswaschbar]

Please share your idea. This bug is worst because it is a major one and I have no clue where the error comes from.

I can basically think of three possibilities:
- the script from one of the player's lobby is wrong, which messes up all the other playernumbers
- one of the clients don't read his playernumber correctly from the script and / or changes it afterwards and send a wrong number to the server
- the server messes up with the numbers

[ILMTitan]

The latest bug comment

In CNetProtocol::InitClient and CNetProtocol::InitLocalClient,
CNet::InitClient is called with wantedNumber, and stores what the number ended up being in myNum. It then calls SendAttemptConnect with wantedNumber. Should that be myNum instead?

[Auswaschbar]

It got changed earlier, but CNet::InitClient should return wantedNumber when it is run in client mode, so it shouldn't make any difference.

[Peet]

I haven't seen this since I updated TASClient to an after-.76b1 beta version and/or SpringLobby...I'm beginning to think it was just a tasclient bug.

[LordMatt]

My observation of this bug is that it only happens to me if I join the host's game very late, i.e. I was away when the game was launched and there was a password prompt I need to type in sometimes to start spring that I am late typing.

[NOiZE]

not sure if this is any help, but when you have this bug, you actually have to rehost the the game, be4 the bug will go away. Just relaunching isn't sufficient.

[Peet]

not sure if this is any help, but when you have this bug, you actually have to rehost the the game, be4 the bug will go away. Just relaunching isn't sufficient.

Hmmm, whenever it happened for me it worked after a few tries.

[Satirik]

when you have that bug make a backup of your script.txt and ask to one of the player having a wrong number to do so and post them here

[KDR_11k]

I usually made it go away by assigning ids so there are no gaps.