4d5a5175c6
Remove duplicate/undefined function
...
Unused and no function definition under that name, the more descriptive name is used instead.
4 years ago
b77686213a
🐛 Correctly wipe temp data from saves
...
Fixes #3167
4 years ago
7e230df94a
Allow forcing Hellfire with command-line flags
...
This is useful for systems such as batocera.linux that have dynamic
launcher items that depend on the presence of the appropriate MPQ files.
4 years ago
7d91bc461a
Improve asset file opening
...
Avoid wrapping asset files via StormLib, open them directly via SDL
instead.
4 years ago
84e73e9a53
Match correct generated names in tests
4 years ago
116d3301f8
Load font during tests that depends on it
4 years ago
eb3012cc0d
Reset player object before unpacking hero data
4 years ago
b4f699f0a3
Validate item network messages
4 years ago
96451ec6ce
Update spawn tests
5 years ago
a985df1242
Align formatting
5 years ago
e261de18f0
Update hellfire unpack tests to recognise full staff name
5 years ago
f7e0c31d8d
Update item tests to handle staff name prefix changes
...
#3039 changed the way staff variants were named but the item tests were expecting the old logic which called them all "Staff of".
5 years ago
379a27b64a
♻️ Convert 'ItemType' enum to enum class
...
🎨 Rename 'ItemType' elements to enum class standards
5 years ago
776a3d4e84
🐛 Use 'Spell' instead of 'RSpell' for clearing used inventory scrolls
...
Readied spell can be swapped to something else in between the cast and the hit frame, resulting in incorrect removal.
This is how currently belt scrolls are checked so we are now using a consistent strategy.
5 years ago
9ef77cb428
Use scoped enum for Direction16.
...
Allows renaming the members to share names with Direction members when
representing the same conceptual direction.
5 years ago
b12de6fe12
Use scoped enum for Direction
...
Identified and removed an instance of Direction being used as an argument for a bool parameter
Removed a single-use temporary variable being cast from sprite frame to direction to size_t
Co-authored-by: Anders Jenbo <anders@jenbo.dk>
Fix alignment of WalkSettings array
5 years ago
b74feb01dc
🐛 Correctly check if item name will fit in the info panel
5 years ago
2d47c12515
Fix charpanel always showing player 0 stats
5 years ago
1b3381ce8e
Move CrawlNum to CrawlTable and make it extern visible
5 years ago
0d3d4c14f6
🚚 Rename 'Dead' concept to 'Corpse' everywhere ( #2808 )
5 years ago
7841c4731e
🚚 Rename 'PkItemStruct' to 'ItemPack'
5 years ago
bb2dfdfb73
🚚 Rename 'ItemStruct' to 'Item'
5 years ago
28134d099c
🚚 Rename 'PkPlayerStruct' to 'PlayerPack'
5 years ago
05d206720a
♻️ Replace 'icursW' and 'icursH' globals with single 'icursSize'
5 years ago
317ebcdd3e
♻️ Replace 'icursW28' and 'icursH28' globals with single 'icursSize28'
5 years ago
a469f7906e
♻️ Replace 'cursW' and 'cursH' globals with single 'cursSize'
5 years ago
bbf529c65a
🚚 Rename 'PlayerStruct' to 'Player'
5 years ago
4044fadeb5
Clean up some players references
5 years ago
3dbc120320
optimized packing/unpacking inventory items ( #2610 )
5 years ago
045921813b
some useless code removal
5 years ago
9217e8c7f0
Test CreatePlayer
5 years ago
82cea78def
Apply clang-format
5 years ago
e3a2142941
Move picosha2.h to 3rdParty
5 years ago
95daecc1bb
Introduce GetSpellListSelection and remove selection game logic from DrawSpellList
5 years ago
90960f0c14
Remove dead code
5 years ago
069249f318
🚚 Move local function to anonymous namespace i-p
5 years ago
70db8456ba
Remove _pwtype and replace it with UsesRangedWeapon
5 years ago
a8ed1998d1
Use Point in path_ functions that reference position
...
In cases like path_get_h_cost this allows simplifying logic where Point provides functions for the intended behaviour in a much simpler to understand package.
This also makes it clearer which functions are const/don't modify the node and which potentially have side effects.
5 years ago
f01d551eb0
Add tests for path_get_h_cost, IsTileWalkable, IsTile(Not)Solid, path_solid_pieces, FindPath
...
Only path_solid_pieces and FindPath are used outside the file, but the other functions are exposed in the header and have behaviour I felt worth testing individually.
5 years ago
0efb134e18
Add tests for quest randomisation code
...
Required splitting the code to a separate function so it can be tested in isolation
5 years ago
77b72da276
♻️ Use monster by reference when possible
5 years ago
352a0ca510
Place local symbols in an anonymous namespace
5 years ago
96bf75b502
Rename player globals
5 years ago
dce78dde62
Rename lighting globals
5 years ago
cb564a2d54
Clean up A-D
5 years ago
ee2e7518c4
Clang-tidy: FunctionCase part 2
5 years ago
cbb84199c2
♻️ Replace Point addition with 'Point + Displacement'
5 years ago
d9e33b36f4
🔥 Delete FileErrDlg as it is now unused
5 years ago
357f6f1dc3
Move RNG functions to their own header/source files
5 years ago
20190ee687
Update random number functions to avoid IB ( #2226 )
...
* Update random number functions to avoid IB
Most calls to set seed were using uint32_t already, there were a few variables which were better served by having their type changed from signed to unsigned. The one exception is ItemStruct::_iSeed. This is an identifier that also happens to be used as a seed occasionally so a comment was added documenting this exception.
Includes suggested style changes for Source/towners.h; Source/towners.cpp; Source/msg.h; Source/multi.h
5 years ago
Andrew James
Anders Jenbo
Gleb Mazovetskiy
Juliano Leal Goncalves
obligaron
qndel