(svn r6283) -Code cleanup: fixed wrong indent in PlayerVehWndProc()

(svn r6282) -Codechange: made a function to create the vehicle list for vehicle list windows
this list is also used by mass goto depot to ensure that they use the same vehicles
right now only the list of all vehicles use this for goto depot, but eventually all the types will use this

(svn r6281) -Fix (r6186): three settings were accidently removed. Thanks go to Osai for reporting and providing the patch.

(svn r6280) -Codechange: Use the same naming for trains as for other vehicles:
CMD_SEND_TRAIN_TO_DEPOT instead of CMD_TRAIN_GOTO_DEPOT

(svn r6279) -Codechange (r1525): Use proper const pointers for functions that do not change them .

(svn r6278) -Revert the colour-mess in the vehicle window for ships, fourwheelers and
aircraft as well which I missed out in the last commit.

(svn r6277) Clean up the train details drawing routine

(svn r6276) Remove a redundant assignment

(svn r6275) Get rid of an ugly cast

(svn r6274) Clean up a bit

(svn r6273) Clean up the custom currency window handler a bit

(svn r6272) -Revert the colour-mess in the vehicle window. Servicing depot destinations
have the same colour as always, if the vehicle goes to stop inside the depot
the colour is red, as it was.
-Tooltip windows should be somewhat short, not a whole essay.

(svn r6271) Remove stale comment

(svn r6270) Remove the "unsorted" vehicle sorter, because it's plain useless

(svn r6269) -Fix r6240: fixed incorrect row count in vehicle lists when resizing to smaller windows (AsterixMG)

(svn r6268) Remove now pointless assignment (followup of r6267)

(svn r6267) Simplify the control flow in the rail tile loop by eliminating a bool flag and replacing it by a simple goto

(svn r6266) Replace four shifts by a single multiplication and let the compiler handle what to do with it - it does a better job

(svn r6265) Check whether to draw catenary at all once before the drawing loop instead of on every iteration

(svn r6264) Get rid of a cast

(svn r6263) WebTranslator2 update to 2006-08-31 12:34:27
dutch - 4 fixed by habell (4)
french - 4 fixed, 4 changed by glx (8)
german - 4 fixed by Neonox (4)
polish - 4 fixed by meush (4)
portuguese - 11 fixed, 17 changed by izhirahider (19), nars (9)
turkish - 9 fixed, 10 changed by jnmbk (19)

(svn r6262) Remove unused #define and function declaration

(svn r6261) -Fix r6246: fixed sprite related crash related to other player's vehicle list windows (spotted by Darkvater)

(svn r6260) Improve readability

(svn r6259) Extract the axis directly, don't use a trinary operator

(svn r6258) Reduce the number of induction variables and inner scopes for clarity

(svn r6257) Use memcmp() to check the MD5 sum instead of using a home brewed loop

(svn r6256) if () cascades -> switch ()

(svn r6255) Simplify drawing of catenary in depots

(svn r6254) if () cascade -> switch ()

(svn r6253) Comment out code with #if 0, not //. Join statements which needlessly span many lines into fewer lines

(svn r6252) Replace a comglomerate of ifs for animated station tiles by a small table and a loop iterating over it

(svn r6251) Replace 2 bool variables by a simple switch

(svn r6249) -Fix: fixed assert when pressing goto depot in an empty list (forgot to disable the button in this condition)
-Code cleanup r6246: simplified SendAllVehiclesToDepot() and moved an { in PlayerVehWndProc()

(svn r6247) -Fix(r6244): invalid arguments caused infinite loop

(svn r6246) -Feature: added the many times requested "send all vehicle to depot" button
it's located in the vehicle list screen and does the same as in the shared orders window (send all vehicles in list to a depot)
it will still not inform the player if a vehicle failed to find a depot, so don't take for granted that all of them go

(svn r6245) -Add: added comments for all YAPF C interface functions (yapf.h) used by other modules, added FollowTrackInit() into YAPF track followers.

(svn r6244) -Fix: allow any order for strgen parameters

(svn r6243) -Codechange: made SortVehicleList() static as well since that one is not used in any other files either (Thanks Tron for pointing this out)

(svn r6242) -Codechange: made BuildVehicleList static as it is not used in any other files anymore
added window_type to arguments and used it to replace an if cascade with a switch case

(svn r6241) -Fix r6240: missed altering one line in last commit

(svn r6240) -Code cleanup: in PlayerVehWndProc: replaced size_of_row with w->resize.step_height
it was only used twice, so there was no reason to calculate it for each event

(svn r6239) -Code cleanup: cleaned up PlayerVehWndProc
code to delete an empty shared orders list is now much simpler
cleaned up the code to handle button clicks
fixed an assert if widget 9 was pressed on a list with vehicles for another company

(svn r6238) Use the colour stored in _colour_gradient[] instead of explicitly retrieving it with GetNonSprite()

(svn r6237) -Fix r6165: fixed wrong hangar name in heading for hangar strings
also fixed inconsistent Danish translation and a typo in Dutch in the same strings

(svn r6236) -Fix (FS#313): TGP generation seed edit box only accepted seeds up to 2^31-1, instead of up to 2^32-2.

(svn r6235) -Cleanup: Make SVN ignore everything not under version control in the data/ folder

(svn r6229) -Feature: Shared order lists now got a "goto depot" button
this will try to send all vehicles in the list to depots/hangars
currently if one fails to find a depot, it will not tell the player

(svn r6228) -Fix: redrawing an empty list of shared vehicles will now close the window
Since the order is no more, the player will not need that particular window anymore
this could happen if the player opened the window and then sold all the vehicles (or crashed them)
A crash could happen if a new order gets the same (now freed) OrderID and the vehicle using it is not of the same type as the window expect

(svn r6227) -Codechange: added window type flags to use with PlayerVehWndProc
this makes the list type detection much easier and allowed an if cascade to be turned into a switch case
this also makes it easier to add more list types

(svn r6226) -Fix(r5864): an assertion was triggered when drawing catenary on steep slopes (TrueLight)

(svn r6225) -Codechange: change the warning/error format of strgen to show up
in MSVC 'Task List' as well.

(svn r6224) WebTranslator2 update to 2006-08-29 22:34:42
dutch - 10 fixed, 14 changed by habell (24)
estonian - 1 fixed, 3 changed by t2t2 (4)
portuguese - 6 fixed, 4 changed by izhirahider (10)

(svn r6223) -Fix: RVs with empty order list were able to enter the depot or road stop from the back. I introduced this bug when I "improved" GetTileTrackStatus(). (thanks glx for the report and fix).

(svn r6222) Remove struct ColorList, because the names of its attributes are plain confusing
All the struct holds is a simple colour gradient, so using a simple array with 8 entries is more clear
Also add the names of colour the gradients as enum

(svn r6220) -Fix: added missing space in a string for Catalan town names (Arnau)

(svn r6215) -Codechange: [vehicle list windows] unified Player(vehicle)WndProc into PlayerVehWndProc
Those 4 unified functions were really much alike, so there was no reason to have so much dublicated code

(svn r6213) -Fix: Calatan town names was only prefix OR the rest of the string, never both (nars)

(svn r6212) Remove dependency on window.h from station.h

(svn r6211) Commit file forgotten in r6210

(svn r6210) Remove FR_NOBORDER, because it is exclusivly used in conjunction with FR_TRANSPARENT

(svn r6209) Move DrawFrameRect() out of gfx.[ch], because it uses data (_color_list) which the renderer shouldn't have know about

(svn r6208) Reduce the number of induction variables for clarity

(svn r6205) -CodeChange : use a switch case instead of a cascade of if
-CodeChange : rename var Engine *e to Engine *eng, since we have WindowEvent *e in same fnct
-CodeChange : fix a broken tabulation on a switch case. SOme breathing spaces too

(svn r6204) -Cleanup: replace non-indentation with spaces; like '}<TAB>else {' -> '} else {', tabs between code and comment, etc.

(svn r6197) WebTranslator2 update to 2006-08-28 14:57:10
american - 10 fixed, 12 changed by WhiteRabbit (22)
portuguese - 4 changed by izhirahider (4)

(svn r6195) -Codechange: do not use '//' in '/* ... */' style comments and '/*' or '*/' in '//' style comments.

(svn r6186) -Fix [FS#309]: add an override system to the dynamically determined size of steps for numeric patch settings in the 'Configure patches' GUI.
The size of these steps normally about (max - min) / 50, which results in steps of 100 000 years for the start year, end year, network restart year and the year of introduction of coloured newspapers.

(svn r6185) WebTranslator2 update to 2006-08-28 10:43:48
danish - 36 fixed, 12 changed by trenskow (48)
french - 2 changed by glx (2)
italian - 36 fixed, 5 changed by sidew (41)

(svn r6184) Remove the unused (because it was NULL in all callers) second parameter of FillDrawPixelInfo() and simplify some expressions

(svn r6183) Move GetDrawStringPlayerColor() out of gfx.[ch]

(svn r6182) -Code cleanup: replaced one (1 << 8) with SHARE_FLAG (missed in r6167)

(svn r6181) -Fix r6161: The list of vehicles using a certain station got a bit of display code mixed up with the list of vehicles with shared orders (nycom)

(svn r6180) Use a switch instead of function pointers to choose the blitter

(svn r6179) - Silence the old npf warnings

(svn r6178) -Fix r5886: only setting ->money64 isn't enough, as some routines still use ->player_money.
Because of this, when a player went bankrupt, still not all stuff was removed.

(svn r6177) WebTranslator2 update to 2006-08-27 22:29:43
czech - 70 fixed, 12 changed by Hadez (82)
german - 4 changed by Neonox (4)
italian - 56 fixed, 8 changed by sidew (64)
portuguese - 2 fixed by izhirahider (2)

(svn r6176) -Fix r6052: I assumed an typedefd enum would be an uint8, which is a really stupid thing to assume.
This caused nasty endian bugs. Thank you SO MUCH Osai for all your time in debugging this!! And tnx Rubidium for spotting :)

(svn r6175) WebTranslator2 update to 2006-08-27 17:18:26
french - 10 fixed, 12 changed by glx (22)
german - 1 fixed by Neonox (1)
polish - 10 fixed, 12 changed by meush (22)
portuguese - 11 fixed, 8 changed by izhirahider (19)

(svn r6174) -Cleanup: [YAPF] some SDT_CONDVARs replaced by SDT_VARs since they are not saved in savegame (Darkvater)

(svn r6173) WebTranslator2 update to 2006-08-27 14:08:35
danish - 35 fixed by trenskow (35)
german - 9 fixed, 8 changed by Neonox (17)
hungarian - 10 fixed, 12 changed by miham (22)
portuguese - 9 fixed by izhirahider (9)
slovak - 2 fixed by lengyel (2)
turkish - 1 fixed by jnmbk (1)

(svn r6170) -Feature: save max_companies/clients/spectators in the config file.

(svn r6169) -Codechange: Use GetString() instead of GetStringWithArgs() which should be
integral to strings.c

(svn r6168) -Code cleanup r6161: removed a newline and added a tab. Thanks TrueLight

(svn r6167) -Code cleanup r6161: replaced some uint16 with OrderID and (1 << 8) with SHARE_FLAG

(svn r6166) -Fix: [YAPF] fixes one very improbable assert when adding startup node that already exists in the open-list (thanks Panzerfather)

(svn r6165) -Feature: control click Goto Depot will now make the vehicle service
at the depot and leave right away.
To tell the difference the status of stopping vehicles will be in red,
while servicing vehicles will be green.
-Codechange: remove some dead code in CmdSendAircraftToHangar() since it is
conflicting with new functionality. Now p2 means the same for all types

(svn r6164) -Fix [YAPF]: Road vehicles preferring bends - added yapf.road_curve_penalty setting with default value 100 (=YAPF_TILE_LENGTH) that is saved only in MP games (mart3p)

(svn r6163) -Codechange: distance rounded up (correction of r6162)

(svn r6162) -Fix: Multistop problem caused by incorrect distance returned by YapfRoadVehDistanceToTile() (by mart3p)

(svn r6161) -Feature: List of vehicles with the same shared orders
the list is available in the orders window and looks like the list buttons from the station windows (small vehicle)
The button is disabled if the vehicle do not have any shared orders or it got shared orders, but an empty order list
based on a patch by nycom, thought it ended up getting heavily modified
Thanks to TrueLight for proofreading and suggestions

(svn r6160) -Fix [ 1519167 ] Bus trying to service in depot of other company (mart3p)
thanks Darkvater for hotfix (r5897)

(svn r6159) -Fix: FindClosestTrainDepot hardly ever found a depot with NPF off due to absence of distance-normalization (Rojer)

(svn r6158) -Revert r6139 as that breaks strings with cases. Thanks michi_cc

(svn r6157) -Codechange: DeleteVehicle removes a vehicle from the pool
-Codechange: DestroyVehicle is called by DeleteVehicle to remove all things where a vehicle depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6156) -Codechange: DeleteIndustry removes an industry from the pool
-Codechange: DestroyIndustry is called by DeleteIndustry to remove all things where a industry depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6155) -Codechange: split Destroy routine from SpriteGroupPoolCleanBlock
Last change is to prepare for new pool system.

(svn r6154) -Cleanup: ordered the IDs that have to do with pools from the others

(svn r6153) -Codechange: DeleteSign removes a sign from the pool
-Codechange: DestroySign is called by DeleteSign to remove all things where a sign depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6152) -Codechange: renamed GetNumRoadStops to GetNumRoadStopsInStation as it reflects its function more

(svn r6151) -Codechange: DeleteStation/DeleteRoadStop removes a station/RoadStop from the pool
-Codechange: DestroyStation/DestroyRoadStop is called by DeleteStation/DeleteRoadStop to remove all things where a station/RoadStop depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6150) -Codechange: introduced and used RoadStopID where needed

(svn r6149) -Codechange: DeleteTown removes a town from the pool
-Codechange: DestroyTown is called by DeleteTown to remove all things where a town depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6148) -Fix r6146: mostly it is useful to declare a variable in the function you use it (tnx Tron)

(svn r6147) -Fix: I don't know how to write Destination :(

(svn r6146) -Fix: MSVC doesn't know how to cast to an union.. so fix it via an indirect (and btw the old) way

(svn r6145) -Codechange: added OrderID to indicate Order indexes out of the pool

(svn r6144) -Codechange: renamed OrderID to VehicleOrderID, because it had nothing to do
with the Order-pool, but with the place of the order within the vehicle-order
(hence its name) (part of FS#13, blathijs)

(svn r6143) -Codechange: DeleteWaypoint removes a waypoint from the pool
-Codechange: DestroyWaypoint is called by DeleteWaypoint to remove all things where a waypoint depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.

(svn r6142) -Codechange: added WaypointID (sorry DV, couldn't splits it anymore)
-Codechange: introduced DestinationID, which is in fact an union of several types
Used in Order struct, so no longer StationID is abused for all targets.
Hangars are a big exception, as they use a station-id with GOTO_DEPOT (go figure)

(svn r6141) -Codechange: introduced DepotID and used it as much as possible
-Codechange: DeleteDepot removes a depot from the pool
-Codechange: DestroyDepot is called by DeleteDepot to remove all things where a depot depends on.
Last 2 changes to prepare for new pool system. Not pretty now, will be soon.
-Codechange: Removed DoDeleteDepot as it was stupid

(svn r6140) -Codechange: fix the comment of _shift_pressed. It doesn't react to ALT, but to SHIFT.

(svn r6139) -Codechange: strings are StringID. This should be changed in strings.h as well, but you would need to include openttd.h and that opens a whole new can of worms.

(svn r6138) - Remove stdafx.c since we do not use it.

(svn r6137) -Codechange: some very minor cleanups:
- Start using DeleteXXX for every pool item, not manually doing it
- Use some wrapper to improve logic
- Rewrote some pieces to improve logic

(svn r6136) -Fix: _cur_year wasn't converting when reading pre-31 savegames, causing nasty desyncs
-Fix: _date, which is a Date, which is int32, was saved as uint32

(svn r6135) WebTranslator2 update to 2006-08-26 09:44:54
dutch - 28 fixed by habell (28)
german - 5 changed by Neonox (5)
portuguese - 2 fixed by izhirahider (2)

(svn r6131) -Codechange : Complete all missing _ttdpatch_flags entries
-Feature : both unifiedmaglevmode are now set.
Maglev and monorail are not allowed to run on each other tracks and will not be.
Setting those flags will allow grfsets as the Norvegian one to be loaded
-Codechange : link the TTDPatch's irregularstations with OTTD's nonuniform_stations
-Codechange : Reformat the whole array (thanks Rubidium, it sure looks better now)

(svn r6122) Fix an off-by-two error in r6108: it should be the index for the last element in the array

(svn r6117) -Fix r6097: on mingw it is normal that both / as \ are accepted as input
-Fix r6096: detect for PATHSEP on WIN32 or WIN64, not UNIX
(Windows compilers define either one, while other compilers don't define a thing)