Skip to content

General Update and Prepare Feature Support - #116

Merged
ergo720 merged 11 commits into
Cxbx-Reloaded:masterfrom
RadWolfie:general-update
Jul 14, 2026
Merged

General Update and Prepare Feature Support#116
ergo720 merged 11 commits into
Cxbx-Reloaded:masterfrom
RadWolfie:general-update

Conversation

@RadWolfie

Copy link
Copy Markdown
Member

Some modifications have been made to either improve the code or add planned feature support.

  • Add TEST_SKIP_OPT and TEST_SKIP_END macros for future feature support.
    • TEST_SKIP_OPT is intended for scenarios where a portion of the tests cannot be executed due to certain conditions.
    • TEST_SKIP_END is intended for scenarios where entire tests cannot be executed under certain conditions, such as when a retail kernel does not export every symbol while a dev/debug kit does.
  • Add an nxdk_patch header file for changes that need to be upstreamed. It is best to document what is missing from the nxdk repository here, then submit pull requests upstream to update the nxdk repository.
  • Improve the TEST_END macro by integrating a return statement, removing the need for redundant return statements in individual tests.
  • Improve the Makefile by:
    • Scanning the .c files only once instead of repeatedly scanning for every file compilation.
    • Fetching the Git version once rather than during every single file compilation.
    • Setting the compilation flags once so they do not repeat for every file compilation.
    • Minor formatting adjustments for better readability.
  • Fix various coding format mistakes discovered along the way.
  • Pulled a couple of PatrickvL's commits from Flesh out tests for all 379 API's #112 pull request:
    • Cherry-picked update device dummy comment.
    • Cherry-picked modified macro argument parenthesization.

@RadWolfie
RadWolfie marked this pull request as draft July 6, 2026 15:31
@RadWolfie

RadWolfie commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Marked as a draft pull request for any review remarks that need to be changed or for any additional changes required.

The configuration variable changes will be created in a separate pull request in the future.

@RadWolfie
RadWolfie marked this pull request as ready for review July 10, 2026 13:03
@ergo720
ergo720 merged commit 5131f49 into Cxbx-Reloaded:master Jul 14, 2026
1 check passed
@RadWolfie
RadWolfie deleted the general-update branch July 14, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants