Skip to content

refactor(memory): Move Memory Management Macros to C++ Template Funct… - #844

Open
MathewBensonCode wants to merge 1 commit into
JeanPhilippeKernel:developfrom
MathewBensonCode:buildfix
Open

MathewBensonCode wants to merge 1 commit into
JeanPhilippeKernel:developfrom
MathewBensonCode:buildfix

Conversation

@MathewBensonCode

Copy link
Copy Markdown
Contributor

…ions

  • This change targets the macros in the ZEngineDef.h header.
  • Refactored the Memory Management Macros to make them use C++ Template Based Functions instead.
  • This will make the code easier to read, especially when there is an issue(error messag) as tracking down what is going on with macros can be quite difficult.
  • The functions have also been made inline so there should be less binary code repetition, which should lead to a smaller binary.
  • This is similar to fix: Header Re-Organization for Consistency and Readability #497 in terms of the number of files touched, but the foundation is in the ZEngineDef.h header...So it may take some effort to review. However, I feel it is a meaningful change that will make implementing modules easier,(among other benefits)

…ions

- This change targets the macros in the `ZEngineDef.h` header.
- Refactored the Memory Management Macros to make them use C++ Template
  Based Functions instead.
- This will make the code easier to read, especially when there is an
  issue(error messag) as tracking down what is going on with macros can be quite difficult.
- The functions have also been made inline so there should be less
  binary code repetition, which should lead to a smaller binary.
@JeanPhilippeKernel JeanPhilippeKernel added enhancement New feature or request area-linux Work on Linux system area-window Work on Window system area-macOS Work on macOS system labels Oct 1, 2026
@JeanPhilippeKernel JeanPhilippeKernel added this to the Future milestone Oct 1, 2026
@JeanPhilippeKernel
JeanPhilippeKernel self-requested a review October 1, 2026 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-linux Work on Linux system area-macOS Work on macOS system area-window Work on Window system enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants