Use player_t for argument to allied
Remove meaningless error code on bad command
Mark help function as static
constify GetPlayer, use player_t instead of int, push declarations closer to definitions
Do address sanitizer by default.  There's no reason not to do this for now.
Pack commands into argv as well as global args/argn.  Add this as a parameter to CommandFunction.  Define command_t as the vector of strings for this.  Update relation to match this signature, and use this vector instead of the global variables.  Existing functionality is preserved for current functions.