Age | Commit message (Collapse) | Author |
|
arrays to _industry_sections
|
|
coordinates of the tile to operate on - to a TileIndex
Remove DoCommandByTile(), because now it does the same as DoCommand()
|
|
|
|
|
|
by TILE_HEIGHT. Reverted one change from the previous commit because it was faulty
|
|
|
|
|
|
Although it was an accessor, nor his usage nor the values were clear.
|
|
variables instead of using the general byte-type.
|
|
|
|
|
|
create/use helper macro/enum for recoloring scheme
|
|
|
|
|
|
the industry from a tile
|
|
|
|
|
|
|
|
is wrong
This leads to graphical glitches when drawing foundations.
This doesn't fix all problems, but at least some of them.
|
|
reservation in savegames and update where used
- Also add this capability to settings
|
|
parameter. (sulai)
|
|
DiagDirection and friends
|
|
|
|
|
|
SL_MAX_VERSION
|
|
|
|
and density of clear tiles with symbolic names and accessors.
See clear.h for details
|
|
|
|
|
|
|
|
|
|
|
|
|
|
-Indentation
-Whitespace
-DeMorgan's Law
-Test with NULL or 0 for non-booleans
-'\0' instead of 0 for chars
-Remove redundantly redundant comments (like DoFoo(); // Do foo)
-Join multiple short lines with a single statement
-Split single lines with multiple statements
-Avoid assignments in if
|
|
|
|
- CHANCE*() instead of mumbling strange numbers
|
|
|
|
about accepted goods of industry tiles, instead of magic numbers
|
|
- const, whitespace, indentation, bracing, GB/SB, pointless casts
- use the trinary operator where appropriate
- data types (uint[] -> AcceptedCargo, ...)
- if cascade -> switch
- if (ptr) -> if (ptr != NULL)
- DeMorgan's Law
- Fix some comments
- 0 -> '\0', change magic numbers to symbolic constants
|
|
comments, excess empty lines, minor other changes
nothing spectacular, just some stuff, which piled up
|
|
|
|
|
|
int/uint/int16/byte/-1 with proper types and constants
|
|
|
|
|
|
proper types or just removing them
|
|
CT_INVALID/OWNER_SPECTATOR/INVALID_STATION where appropriate
|
|
functions that don't change their pointer parameters
- change a lot of byte player types to PlayerID
- beautify header files, same "#endif /* filename */" ending
|
|
owner it is accessed by [GS]etOwner anyway and when it doesn't store an owner, but arbitrary data, accessing a field called "owner" is confusing.
|
|
strings more explicit
|