devilutionX

Commit Graph

Author	SHA1	Message	Date
Anders Jenbo	bc59218d0d	More clang-tidy/Android Studio cleanups	5 years ago
Gleb Mazovetskiy	55464eec78	Fix `-Wmissing-braces` Fixes warnings such as this one: ``` ../../../../../../Source/engine/render/text_render.hpp:83:33: warning: suggest braces around initialization of subobject [-Wmissing-braces] return DrawString(out, text, { position.x, position.y, out.w() - position.x, 0 }, flags, spacing, lineHeight, drawTextCursor); ^~~~~~~~~~~~~~~~~~~~~~ ```	5 years ago
Anders Jenbo	a7c7fa0030	Fully apply clang-tidy/format to all files	5 years ago
Anders Jenbo	ecea12fc15	Clang-tidy: ParameterCase	5 years ago
Anders Jenbo	ee2e7518c4	Clang-tidy: FunctionCase part 2	5 years ago
Gleb Mazovetskiy	cb0dae8590	🚚 Rename `CelOutputBuf` to `Surface` and extract	5 years ago
Anders Jenbo	a350fb0b15	Clang-tidy: FunctionCase part 1	5 years ago
Anders Jenbo	4cd916a085	Clang-tidy: EnumConstantCase	5 years ago
Anders Jenbo	e426c38adb	Correct AddInitItems	5 years ago
Anders Jenbo	d9e0658643	Clang-tidy: basic naming	5 years ago
Anders Jenbo	b1d237c823	More init clean up (#2262 )	5 years ago
Juliano Leal Goncalves	cbb84199c2	♻️ Replace Point addition with 'Point + Displacement'	5 years ago
ephphatha	357f6f1dc3	Move RNG functions to their own header/source files	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	8a4d38dd74	Stabalize clang-tidy runs	5 years ago
Anders Jenbo	f167bd4dfb	Clang-tidy: llvm-include-order	5 years ago
Anders Jenbo	de4ced7e4f	Clang-tidy: readability-qualified-auto	5 years ago
Anders Jenbo	f4f8f22699	Clang-tidy: readability-redundant-control-flow	5 years ago
Anders Jenbo	bf221f4322	Clang-tidy: readability-simplify-boolean-expr	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
Anders Jenbo	1cc0eb25ca	♻️Use ranges for loops (clang-tidy)	5 years ago
Anders Jenbo	625333dfb1	♻️Clean up items	5 years ago
Andrew James	60a47caf1b	Refactor Draw* functions to use Rectangle and Point types Added overload for DrawString taking a Point to avoid creating a rect for callers which only use position. This also documents the way DrawString operates when passed a clipping rectangle with a dimension of 0. As part of this overload removed the logic for 0 width regions from DrawString. This does change the behaviour of the Rectangle version if called with a rect with width 0, all callers using that behaviour have been updated in this commit. Using Rectangle/Size allowed simplifying the logic for certain calls where they could use DrawText alignment flags, previously this was manually aligning by calculating dimensions and offsetting the position. This also fixes #2169 Also includes a few instances where a temporary buffer was used to set the text to be drawn with unbounded sprintf calls, replaced those with snprintf as is recommended in modern C applications. Moving to C++ strings would be good in a future refactor.	5 years ago
Anders Jenbo	ee7b32d8f6	Player by reference part 3	5 years ago
qndel	92d1006134	Use std::move for premiumitems shift	5 years ago
Juliano Leal Goncalves	604b72bb12	Reword potion effect descriptions for consistency	5 years ago
Gleb Mazovetskiy	e40b1963f9	🚚 engine.h: Extract `CelSprite` and file loading	5 years ago
Gleb Mazovetskiy	dcf3397139	🚚 engine.h: Extract `ActorPosition`	5 years ago
Juliano Leal Goncalves	d47fc924f3	♻️ Use '_item_indexes' instead of 'int' for 'ItemStruct.IDidx' field	5 years ago
obligaron	ab3f3f3ee7	Introduce ItemStruct.SetNewAnimation(bool) to start Animations only at one location	5 years ago
obligaron	6d5291b17d	Enable ADL for Items	5 years ago
obligaron	ccc1ccf9c3	Introduce ItemStruct.AnimInfo	5 years ago
Anders Jenbo	27591cc29f	Fix spawn save game compatability	5 years ago
Juliano Leal Goncalves	e67683e5be	♻️ Simplify 'GetSuperItemLoc' logic using 'Point'.	5 years ago
Juliano Leal Goncalves	b0584f1925	♻️ Simplify 'GetSuperItemSpace' logic using 'Point'.	5 years ago
Juliano Leal Goncalves	1cc121e95c	♻️ Simplify 'SpawnRock' logic using 'Point'.	5 years ago
Juliano Leal Goncalves	1bf319bc11	♻️ Leverage 'Point' struct on 'ItemPlace'	5 years ago
Juliano Leal Goncalves	6aa6efb720	♻️ Extract item position generation logic to common function	5 years ago
Juliano Leal Goncalves	b04047ba4d	♻️ Leverage 'Point' struct on 'SpawnUnique'	5 years ago
Juliano Leal Goncalves	9c85e61025	♻️ Leverage 'Point' struct on 'ItemSpaceOk'	5 years ago
qndel	ab0478319c	assign seed to quest items + rng cleanup	5 years ago
Kalebe Alves	118efe8e6d	Use Point/SDL_Rect for control functions coordinate system	5 years ago
Gleb Mazovetskiy	dd3ea66fa2	♻️ Split getTextColor(bool) into 2 functions This is a bit more readable than using a bool argument.	5 years ago
qndel	2f3cf65b27	cleanup getTextColor	5 years ago
qndel	b1d74d2791	alt highlighting (#2060 )	5 years ago
Anders Jenbo	68249d07e3	Correct false positives translation warnings Fixes #2053	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

1 2 3 4 5 ...

688 Commits (e1e92ad77741472bb1dcf8e076efc8fc274c2f6e)