Anders Jenbo
0f2ad469ea
♻️ Clean up variable initialization
5 years ago
Anders Jenbo
8e07ef8fe5
Clang-tidy: misc-unused-parameters
5 years ago
Juliano Leal Goncalves
fd9f2099d0
♻️ Leverage 'Point' to represent mouse position
5 years ago
ephphatha
357f6f1dc3
Move RNG functions to their own header/source files
5 years ago
obligaron
3a9cc631c7
Rename DelayLen -> TicksPerFrame
...
Rename DelayCounter -> TickCounterOfCurrentFrame
5 years ago
obligaron
3ba4bac5e2
Harmonize DelayLen for Player and Monster (was 1-based on monseter (>= DelayLen) and 0-based on player (> DelayLen). Now both are 1-based (>= DelayLen)
5 years ago
Anders Jenbo
2fdbd39bb0
Clang-tidy: modernize-raw-string-literal
5 years ago
Anders Jenbo
bf780be1ed
♻️ Cleanup all implicit bool conversions
5 years ago
Anders Jenbo
8f3266244a
Clang-tidy: readability-else-after-return
5 years ago
Anders Jenbo
790f42d078
Clang-tidy: readability-inconsistent-declaration-parameter-name
5 years ago
Anders Jenbo
19d980582f
Clang-tidy: readability-static-definition-in-anonymous-namespace
5 years ago
Anders Jenbo
2bdb6caa07
Cleanup majority of remaning implicit bool conversions ( #2234 )
5 years ago
ephphatha
f4f6a8a51e
Refactor PlaySfxLoc to take a Point instead of x/y params
...
Because of the heavy usage of this function in objects.cpp and the mixed use of aliases and direct references to the active object in Operate*Door functions I've done some refactoring of those areas beyond what is strictly necessary. Hopefully this makes sense and is reasonable to include in this change.
5 years ago
Anders Jenbo
d8fa29c972
Clean up more implicit bool conversion
5 years ago
Anders Jenbo
22b632f04a
♻️ Clean up implicit bool conversions
5 years ago
obligaron
317a6ee82c
Introduce MonsterStruct::Petrify
5 years ago
Anders Jenbo
1cc0eb25ca
♻️ Use ranges for loops (clang-tidy)
5 years ago
Anders Jenbo
35b2aa4947
♻️ Unfold a few functions
5 years ago
Anders Jenbo
e008d66f14
♻️ Declare type with assignment
5 years ago
Anders Jenbo
ee7b32d8f6
Player by reference part 3
5 years ago
Gleb Mazovetskiy
8a9618aea2
♻️ Clean up `cel_header.hpp`
...
1. Rename `CelGetFrameStart` to `CelGetFrame`, in line with the other 2
functions with the same name and load the `uint32_t` safely.
2. Remove redundant `FrameHeader`, simply use `LoadLE16`.
3. Document all the functions.
5 years ago
Matheus Gomes
5ee7edbde1
Refactored "StartWalk(...)" into different functions and moved tightly coupled "Direction" parameters into an array.
5 years ago
Vladimir Olteanu
95643029e3
Refactor missiles.cpp
...
Use Point wherever possible
Make some functions static (not exhaustive)
Reduce excessive indentation (not exhaustive)
Declare variables as needed (not exhaustive)
5 years ago
Gleb Mazovetskiy
14218080b6
🚚 engine.h: Extract `CelGetFrame` and friends
5 years ago
Gleb Mazovetskiy
e40b1963f9
🚚 engine.h: Extract `CelSprite` and file loading
5 years ago
Anders Jenbo
c6f6c99add
Fix GCC warning
5 years ago
qndel
816b244689
HasItem/RemoveInvItem combo refactor
5 years ago
Juliano Leal Goncalves
9c85e61025
♻️ Leverage 'Point' struct on 'ItemSpaceOk'
5 years ago
Gleb Mazovetskiy
f319c593a6
🧹 player.cpp: Remove unused `GetPlrGFXClass`
5 years ago
Juliano Leal Goncalves
dd9e05222d
♻️ Change 'PosOkPlayer' to take a 'Point' instead of 2 separate ints
5 years ago
Juliano Leal Goncalves
c6fafbf64b
♻️ Change 'SolicLoc' to take a 'Point' instead of 2 separate ints
5 years ago
Anders Jenbo
8a7ec56d3a
Player by reference part 2
...
This mostly change player to be by reference instead of by index.
But additionally it does stript checks for gold in the belt, move some
value types to the initialization and short circute a few functiongs.
5 years ago
obligaron
da7ded3170
Integrate GetAnimationWidth in LoadPlrGFX (only callside)
5 years ago
obligaron
d97131349e
Unify player_graphic memory handling
5 years ago
obligaron
61ba5009ac
Convert to CelSprite
5 years ago
obligaron
3067aeacb8
Remove PlayerStruct._pXYZWidth and replace it with calls to GetAnimationWidth
5 years ago
obligaron
d11ed50585
NewPlrAnim now also loads the graphics if needed
5 years ago
obligaron
c424f601ec
Change LoadPlrGFX from pnum to PlayerStruct
5 years ago
obligaron
3a6e631261
Remove memset for PlayerStruct
5 years ago
Juliano Leal Goncalves
b3ec79af4f
Leverage 'Point' in Lighting and Player functions ( #2048 )
5 years ago
Juliano Leal Goncalves
cc936dacd7
♻️ Replace 'int' with 'Direction' parameter on 'PlrDirOK'
5 years ago
Anders Jenbo
6ce8f13751
Apply clang-tidy to more code and do some related cleanups
5 years ago
Juliano Leal Goncalves
21fdb0dcd9
♻️ Replace manual direction displacement logic with 'Point' add operator
5 years ago
Juliano Leal Goncalves
d55cd99dd5
🚚 Rename 'direction' enum to 'Direction'
5 years ago
Juliano Leal Goncalves
15c46b9eb0
💬 Rename HeroSpeech members to improve discoverability
5 years ago
Juliano Leal Goncalves
5350250a76
🚚 Rename speech functions to 'Say' variations
5 years ago
Juliano Leal Goncalves
8c620ce29e
♻️ Extract 'HeroSpeech' enum class for hero sounds instead of using raw `int`s
5 years ago
Jmgr
bbd39aa513
Replace sprintfs with strcpy for translatable strings
5 years ago
Jmgr
6e85cde978
Update format strings
5 years ago
Jmgr
0c27e756f7
Add fmt::format
5 years ago