Id master getservers request about etlegacy

[ETLegacyTeam]

From: acqu (acqu)
Date: 2012-05-10T03:06:58Z

---

After reviewing this again i think it is better to handle this in ui code. The engine basically expects the string from ui code, so it is probably more appropriate to have the fix in ui. The places to go for here are in UI_StartServerRefresh, and there you need to change this

@trap_Cmd_ExecuteText( EXEC_APPEND, va( "globalservers %d %s\n", 0, ptr));
trap_Cmd_ExecuteText( EXEC_APPEND, va( "globalservers %d %d\n", 0, (int)trap_Cvar_VariableValue( "protocol" ) ) );@

to

@trap_Cmd_ExecuteText( EXEC_APPEND, va( "globalservers %d %s empty full\n", 0, ptr));
trap_Cmd_ExecuteText( EXEC_APPEND, va( "globalservers %d %d empty full\n", 0, (int)trap_Cvar_VariableValue( "protocol" ) ) );@

this. Thoughts ?

[ETLegacyTeam]

From: @IR4T4 (IR4T4)
Date: 2012-05-10T05:02:07Z

---

Done. Good catch to see empty servers and get them populated. About the full servers I'm not sure about because you can't connect for real here.

Note I: Keep in mind MAX_DISPLAY_SERVERS is 4096 :D ....
Note II: Should we exclude ETPro servers in ET:L ? Doesn't make sense to list unsupported mods (unfortunately).

[ETLegacyTeam]

From: acqu (acqu)
Date: 2012-05-10T15:21:55Z

---

For note 1: 2.6b serverlist lists curretnly about 1000 servers. 2.55 even less :P. So i think this is not a problem, because et is dying anyways and only devs and hardcore gamers remain, which will probably never in future reach 4000+ servers. I think there is even a backup method, at least i can remember i saw it somewhere. Full servers should be listed as well, no need to hide information if there is not a really good reason. E.g. battlefield 3 game browser lists also full servers, why would he not list them O_O

For note 2. Hard one, and this won't be my decision :P I think i would leave it like it is and assume a mature game user :P