Mercurial > hg > openttd
diff src/command.cpp @ 10237:d39d46dc7057 draft
(svn r14465) -Codechange: minor code style of command.cpp.
author | rubidium <rubidium@openttd.org> |
---|---|
date | Tue, 14 Oct 2008 18:49:21 +0000 |
parents | fe1681c9d05b |
children | 45ca88a8de7d |
line wrap: on
line diff
--- a/src/command.cpp +++ b/src/command.cpp @@ -349,7 +349,7 @@ * @param cmd The integervalue of a command * @return true if the command is valid (and got a CommandProc function) */ -bool IsValidCommand(uint cmd) +bool IsValidCommand(uint32 cmd) { cmd &= 0xFF; @@ -364,12 +364,11 @@ * * @param cmd The integer value of the command * @return The flags for this command - * @bug integervalues which are less equals 0xFF and greater than the - * size of _command_proc_table can result in an index out of bounce - * error (which doesn't happend anyway). Check function #IsValidCommand(). (Progman) */ -byte GetCommandFlags(uint cmd) +byte GetCommandFlags(uint32 cmd) { + assert(IsValidCommand(cmd)); + return _command_proc_table[cmd & 0xFF].flags; } @@ -389,7 +388,6 @@ CommandCost DoCommand(TileIndex tile, uint32 p1, uint32 p2, uint32 flags, uint32 procc) { CommandCost res; - CommandProc *proc; /* Do not even think about executing out-of-bounds tile-commands */ if (!IsValidTile(tile)) { @@ -397,7 +395,7 @@ return CMD_ERROR; } - proc = _command_proc_table[procc].proc; + CommandProc *proc = _command_proc_table[procc].proc; if (_docommand_recursive == 0) _error_message = INVALID_STRING_ID;