Skip to content

Conversation

@firewave
Copy link
Collaborator

@firewave firewave commented Nov 3, 2025

  • split Settings from *Options
  • avoid some settings copies
  • removed some redundant/unused parameters
  • cleaned up macros to consistently using varargs
  • moved some implementations to their respective macros
  • added TODOs

@firewave firewave force-pushed the test-options branch 4 times, most recently from 6697db1 to 307ce9a Compare November 6, 2025 16:34
@firewave firewave marked this pull request as ready for review November 6, 2025 16:38
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 6, 2025

@firewave firewave merged commit 7e09fa0 into danmar:main Nov 7, 2025
55 checks passed
@firewave firewave deleted the test-options branch November 7, 2025 14:07
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.

2 participants