Anders Jenbo
a350fb0b15
Clang-tidy: FunctionCase part 1
5 years ago
Anders Jenbo
b71b0149a3
Clang-tidy: ConstexprVariableCase
5 years ago
Anders Jenbo
4cd916a085
Clang-tidy: EnumConstantCase
5 years ago
Anders Jenbo
d9e0658643
Clang-tidy: basic naming
5 years ago
Anders Jenbo
3a1a44de34
Fix and clean up AddDoppelganger
5 years ago
Juliano Leal Goncalves
cbb84199c2
♻️ Replace Point addition with 'Point + Displacement'
5 years ago
Juliano Leal Goncalves
0441ffc2f4
♻️ Move 'fromDirection' from 'Point' type to 'Displacement' type
5 years ago
Anders Jenbo
5537fe748e
Clean up debug code
5 years ago
Anders Jenbo
214cc4b356
Check target before attempting to talk to monsters
5 years ago
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