| Age | Commit message (Collapse) | Author |
|---|
|
correctness.
-Codechange: 'quited' is not a word, use has_quit instead for NetworkClientState
|
|
p->index + 1. To
correctly handle this ci->client_playas - 1 was used all over the code making
it pretty confusing at times. Use proper one-on-one values now. Special handling
is only needed for user-output to not to confuse users.
|
|
windows
|
|
|
|
PlayerID is an
actual playable player (< MAX_PLAYERS) or not.
|
|
PLAYER_SPECTATOR and
put it into player.h where it belongs (instead of map.h)
|
|
IS_INTERACTIVE_PLAYER
|
|
order is a transfer order, rather than to any unload order. This fixes an issue where the AI doesn't know that a route is unprofitable.
|
|
- If the NewGRF sound effect chosen doesn't exist, then ignore it.
- Play load/unload sound if provided.
|
|
Now the num_engines array is updated for the buying player when merging two players
The incorrect caches aren't saved so no savegames will contain incorrect data due to this bug
|
|
more uniform.
-Cleanup: whitespace alignment of a few tables.
|
|
was given
the maximum amount of money so everything can be removed for sure. However
if first rails are removed (or property in general that *gives* money
when demolished, the player's money would overflow. Thanks for noticing Rubidium.
|
|
{' -> '} else {', tabs between code and comment, etc.
|
|
routines still use ->player_money.
Because of this, when a player went bankrupt, still not all stuff was removed.
|
|
a valid XXX, unless there are none to pick from. Then NULL is returned.
|
|
(or, will do that).
It isn't the best name, but we couldn't find any better.
This unifies the pool-system even more.
|
|
invalid ones
-Codechange: use IsValidXXX where ever possible
Note: both changes to prepare for new pool system, which needs those changes.
For every pool there are 2 ugly lines, which will be removed when done
implementing new pool system.
Based on FS#13 by blathijs, partly implemented.
|
|
whitespace fixes only except for a few comments to make them uniform for the whole enum/struct.
|
|
-Cleanup: use TownID and StationID for two instances of uint16
|
|
|
|
the offset since 1920
-Codechange: store all year related variables that are _not_ stored in a savegame/transported over the network in the same format as _cur_year
|
|
BASE_YEAR when comparing _cur_year with a 'full' year.
-Cleanup: replace some magic '1920' values with BASE_YEAR.
|
|
date.[ch]
-Cleanup: fix whitespace related coding style issues in date.[ch]
-Cleanup: make original comments doxygen compatible and remove/change outdated comments
|
|
to lack of money.
|
|
|
|
|
|
DeliverGoodsToIndustry to not work as intended. Note: write something better for this entire function
|
|
serving a station has been deleted.
-Negative side effect: upon loading old games, stations whose last vehicle was a station have (temporarily) lower ratings.
-Positive side effect: station.h no longer includes vehicle.h (breaks the station.h->vehicle.h->rail.h chain)
|
|
coordinates of the tile to operate on - to a TileIndex
Remove DoCommandByTile(), because now it does the same as DoCommand()
|
|
|
|
because the rest of the code calls it Comapany HQ. Moved it to unmovable_cmd.c because CompanyHQ is an unmovable and has nothing to do with economy.
|
|
variables instead of using the general uint16-type. StationID was added for depots, waypoints and stations where necessary. We probably need to change GetDepot(), IsDepotIndex(), IsStationIndex(), GetWaypoint() and IsWaypointIndex() as well to use StationID.
|
|
variables instead of using the general byte-type.
|
|
SL_MAX_VERSION
|
|
|
|
add consts, reduce indentation level by short-circuit logic, convert if cascades to switch, whitespace, bracing, plus some minor stuff
|
|
- Don't treat non-booleans as booleans
- Reduce variable scope
- Bracing
- Use DeMorgan's law to make conditionals easier to read
- if cascade -> switch
- Replace some magic numbers by symbolic names
- Avoid assignments within other statements
|
|
newgrf_engine.[ch], and add the new files to project files.
|
|
|
|
|
|
you full points if you had no vehicles, and was limited to 20k. Solved both. (Partly based on Graphite his patch)
|
|
but gives a bit more fair amount of money. Can only be fixed correctly with CargoPackets as suggested in branch.
|
|
-Fix: the AI speed control is done by the AI-core, individual AIs don't have to do it (so, AIs were delayed twice ;)
-Add: Support for AI-network-clients (an AI, connecting to a remote server)
-Fix: minor AI-core problems
|
|
This means that any user attempt to remove a rear engine will tell the user to move the front engine instead
This fixes the assert when moving multiheaded engines (introduced in r3144)
Note: to make old savegames use this feature, some engines might be turned around in order to link engines in pairs
-Codechange: train subtype is now a bitmask
This allows fast access to info like if it is a wagon or engine and if it is in front and so on
Note: savegame version bump
|
|
|
|
-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
|
|
|
|
the train instead of the number of wagons. The actual length is cached in the first vehicle of the train.
|
|
comments, excess empty lines, minor other changes
nothing spectacular, just some stuff, which piled up
|
|
0/-1 -> INVALID_ENGINE
0x10 -> OWNER_NONE
|
