Some stuff I noticed in Beta 2

[Xpert]

- Need to add an RCON Kill message output for when a base defense kills a player/bot

- There's no RCON crate message output

- There's no RCON purchase message output

- Some kind of game_info command like renegade to show the current map and time

- When you climb a ladder, you can still aim your gun in all directions including behind you.

- After dying, there should be a timer to auto respawn. You can exploit gameplay by just sitting in that one spot and not respawn and you can pretty much spectate whatever is around your dead body to report to your team. This is especially exploitable in clanwars.

- Output for C-LIST to state if there's no players ingame and to also include bots to the list

- Spawned character doesn't have sidearm changes applied. Example, you wanted the machine pistol as your sidearm. You have it applied to your Soldier. You die. When you respawn, your Soldier doesn't have the machine pistol and you have to purchase a new Soldier. This also occurs in character crates. Character crates doesn't have sidearms applied unless that's intentional.

- When using Q on buildings, there is a space between the last word in the sentence and the exclamation mark when it radios "Defend that structure"

- Immediately is spelled wrong in radio command "Defend that structure immediately!"

- Admin command FORCEEND doesn't work anymore. It worked in beta 1.

- Admin command SWAPTEAMS doesn't work anymore. It worked in beta 1.

- An RCON output for when bots join the game.

[Agent]

For RCON, there's also the following missing:

Proper documentation

A way to reliably identify yourself on RCON from other connections (though you can send a bogus command and catch its execution)

Events:

Radio chat

Beacon placement/detonation/disarm.

Airstrike deployment/completion.

C4 detonation/disarm.

Poll creation/completion (with results).

Team change. (You can fairly reliably guess this based on "died by" event)

Building capture.

Edit: Though Kil did say he plans to do far more with RCON once the crashes on map-load are figured out.

[Xpert]

Upon further testing, I find the C-LIST output very annoying. There's really no consistant delimeters.

All the other rcon outputs in beta 2 had a delim of ascii value "160". The C-LIST output only has a standard space (ascii 32) and it doesn't divide any of the information. It's hard to parse the output and tokenize it. It would be great if this gets changed.

If not, I'll just deal with the sloppy if statements I used to keep it consistant.

if ($gettok($2,3,32) == -----NO) && ($gettok($2,4,32) == STEAM-----) { var %steamid = None, %color = $iif($gettok($2,5,32) == GDI,8,4), %team = $gettok($2,5,32), %player = $gettok($2,6-,32) }

else { var %color = $iif($gettok($2,4,32) == GDI,8,4), %steamid = $gettok($2,3,32), %team = $gettok($2,4,32), %player = $gettok($2,5-,32) }

lol

[iran]

if ($gettok($2,3,32) == -----NO) && ($gettok($2,4,32) == STEAM-----) { var %steamid = None, %color = $iif($gettok($2,5,32) == GDI,8,4), %team = $gettok($2,5,32), %player = $gettok($2,6-,32) }

else { var %color = $iif($gettok($2,4,32) == GDI,8,4), %steamid = $gettok($2,3,32), %team = $gettok($2,4,32), %player = $gettok($2,5-,32) }

lol

lol geek

[Agent]

I didn't find C-LIST all that bad to parse tbh, though the space in the "NO STEAM" part was rather annoying. Should just replace it with N/A or something.

Also, from IRC one day:

it's not delimited by the nbsp, because its the same output as what the adminclientlist command does