summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2021-06-06Codechange: move SLF_NOT_IN_SAVE into settingsPatric Stout
It is a settings-only flag, so don't pollute SaveLoad code with it.
2021-06-06Codechange: remove the unused SLF_HEX flagPatric Stout
2021-06-06Update: Translations from eintstranslators
english (us): 1 change by 2TallTyler catalan: 5 changes by J0anJosep
2021-06-06Codechange: Remove FOR_ALL_CHUNK_HANDLERSglx22
Co-Authored-By: Patric Stout <truebrain@openttd.org>
2021-06-05Update: Translations from eintstranslators
spanish (mexican): 1 change by absay latvian: 2 changes by osprinkis
2021-06-04Update: Translations from eintstranslators
korean: 1 change by telk5093 dutch: 1 change by Afoklala french: 1 change by MalaGaM
2021-06-03Update: Translations from eintstranslators
korean: 1 change by telk5093
2021-06-03Fix ef991b17: server was trying to free() a packet created with "new ↵Patric Stout
CommandPacket()" (#9334)
2021-06-03Codechange: rename SettingGuiFlag to SettingFlag (#9332)Patric Stout
It is a lovely organicly grown enum, where it started off with GUI-only flags, and after that a few flags got added that can be considered GUI-only (the GUI disables/enables based on them), to only have flags added that has nothing to do with the GUI. So be less confusing, and rename them to what they do. Additionally, I took this opportunity to rename 0ISDISABLED to reflect what it really does.
2021-06-03Codechange: use "[[maybe_unused]]" instead of a wide variety of other ways ↵Patric Stout
we had While at it, replace OTTD_ASSERT with WITH_ASSERT, as this is always set if assert() is valid. No matter if NDEBUG is set or not.
2021-06-03Codechange: remove unused assert_msg() macroPatric Stout
2021-06-01Update: Translations from eintstranslators
swedish: 1 change by joeax910
2021-06-01Fix b791ffc6: use the correct name in CmdPlantTreeVít Šefl
2021-06-01Fix b791ffc6: use the correct parameter in CmdSignalTrackHelperVít Šefl
2021-05-31Codechange: C++-ify lists for SaveLoad (#9323)Patric Stout
Basically, this changes "SaveLoad *" to either: 1) "SaveLoadTable" if a list of SaveLoads was meant 2) "SaveLoad &" if a single entry was meant As added bonus, this removes SL_END / SLE_END / SLEG_END. This also adds core/span.hpp, a "std::span"-lite.
2021-05-31Update: Translations from eintstranslators
japanese: 1 change by scabtert
2021-05-31Codechange: use AsIntSetting()->Read() wrapper if possible (#9324)Patric Stout
2021-05-30Update: Translations from eintstranslators
korean: 2 changes by telk5093 russian: 1 change by Ln-Wolf finnish: 1 change by hpiirai portuguese: 1 change by azulcosta portuguese (brazilian): 1 change by Vimerum
2021-05-30Fix #9320, 2e136285: nullptr could not be converted to stringRubidium
2021-05-30Fix #9316, 64eddaeb: at about 250.000 inhabitants the bridge length limit ↵rubidium42
check overflows
2021-05-30Codechange: add a wrapper function to find all settings based on prefix (#9312)Patric Stout
2021-05-30Codechange: use setting name instead of index for HandleOldDiffCustom() (#9311)Patric Stout
2021-05-30Codechange: rename _SQ64 into POINTER_IS_64BIT (#9313)Patric Stout
2021-05-30Codechange: [Network] Let admin-console use std::string(_view)rubidium42
2021-05-30Codechange: [Network] Let admin-game script use std::stringrubidium42
2021-05-30Codechange: [Game] Pass the length instead of '\0' terminating (and undoing ↵rubidium42
that) in the middle of a C-string
2021-05-30Codechange: [Network] Make admin name and version std::stringrubidium42
2021-05-30Fix: limit heightmap sizes to something reasonable to prevent crafted ↵rubidium42
heightmaps to OOM-crash the game
2021-05-30Codechange: [Network] Use std::string to determine an unique name for clientsrubidium42
2021-05-30Codechange: [Network] Use std::string to send the client name and rcon commandsrubidium42
2021-05-30Codechange: [Network] Use std::string for server side logic of kicking and ↵rubidium42
banning clients
2021-05-30Codechange: [Network] Let server rcon result use std::stringrubidium42
2021-05-30Codechange: [Network] Let server changing a client name use std::stringrubidium42
2021-05-29Codechange: use setting name instead of index for CmdChange(Company)Setting ↵Patric Stout
(#9306) This is mostly done as there are now constraints on settings.ini you might not expected. For example, conditional settings always have to come last, as otherwise they would influence the index.
2021-05-29Codechange: Rename window related DeleteXXX to match new behaviourglx22
2021-05-29Fix f6d5c01: Delay deletion when closing windowsglx22
2021-05-29Codechange: [Network] Use std::string in CommandPacketrubidium42
2021-05-29Codechange: move from C-string to std::string for DoCommandrubidium42
2021-05-29Codechange: let IsUnique.* functions accept std::stringrubidium42
2021-05-29Codechange: add std::string variant of Utf8StringLengthrubidium42
2021-05-29Fix: Do not send vehicles towards incomplete PF nodesVít Šefl
YAPF could end up in a situation where it sets the best intermediate node to a node whose construction is never finalized (i.e. it is never added to the open list). The content of the node would be overwritten in the next round, potentially sending the vehicle to an unwanted location.
2021-05-29Codechange: move casting a "const char *" to "char *" to the callerPatric Stout
It is not nice to have your definition say you won't change a value while in fact the whole reason for your existance is to change it.
2021-05-29Fix: ScriptObject::DoCommand could modify "text" while defined "const"Patric Stout
This could have unwanted side-effects, as it could change the source for ever and ever.
2021-05-29Doc: describe what each settings-file is aboutPatric Stout
2021-05-29Codechange: move all settings.ini files to table/settingsPatric Stout
2021-05-29Codechange: rename str_validate to StrMakeValid(InPlace) (#9304)Patric Stout
This to be more explicit the function changes the value, and not returns yes/no.
2021-05-29Fix #9281: acquire a company uses special bookkeeping to make you rich (#9300)Patric Stout
When you buy-out a company, you got your shares back. This is based on company-value, which includes values for the vehicles etc. In other words, you not only got the vehicles, but you also got paid to get them back. Additionally, you also got the loan of the company, but not the money for the loan (as that is subtracted from the company-value). Solve this by changing the rules of a buy-out: don't sell your shares, get the loan AND the balance and get the infrastructure.
2021-05-29Cleanup: set the base in the setting templates, instead of defining the base ↵rubidium42
for every setting This has the added benefit of not getting mistaken that multiple bases can be used for the same SettingTable
2021-05-29Fix: [Network] Prevent an empty server name to be set anywhererubidium42
2021-05-29Codechange: use separate pre and post callbacks for int settingsrubidium42