arunkumarmunusamy / bitfighter

Automatically exported from code.google.com/p/bitfighter
Other
0 stars 0 forks source link

Enable the system for removing duplicate servers from the QueryServers UI #380

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
There are 4 lines marked with // ENABLE FOR 020 -- uncomment these lines when 
it is safe to break compatibility with 019 series clients.

Doing so will (hopefully) help identify local servers that are also introduced 
by the master server.  Sometimes these local servers are unpingable, and just 
clog up the server list.  If all goes well, these entries will be removed from 
the server list.

It is difficult to test in the 019a era, so most testing will need to be held 
until 020.

Also consider replacing the S32 clientId with a U32, or possibly reducing the 
S32 to an S16 accompanied by a check to ensure duplicate ids are not issued 
(the check would have to be on the master, and is currently not done due to the 
vastness of an S32 int and the unliklihood of wraparound).

Original issue reported on code.google.com by watusim...@bitfighter.org on 29 Jan 2014 at 11:00

GoogleCodeExporter commented 9 years ago
This has now been enabled for 019 without breaking compatibility to older 019 
in revision aa08524ed2e4

Original comment by sam8641@gmail.com on 30 Mar 2014 at 11:05