Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More C++ tweaks and changes #9478

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

More C++ tweaks and changes #9478

wants to merge 11 commits into from

Conversation

grendello
Copy link
Contributor

@grendello grendello commented Oct 31, 2024

Continued effort to restructure our native C++ runtime, introducing
C++23 features as well as turning as many classes into static ones since
we don't really have objects that need to be created and destroyed during
lifetime of the application:

  • Replace abort_if* preprocessor macros with templated functions. This
    gives us better type safety. Instead using variadic arguments, we now
    provide overloads which take either a plain string without placeholders
    or a lambda function to format the message string on demand.
  • More std::string_view for literal strings
  • More functions decorated with noexcept (this is going to be important
    once exceptions are enabled in the future)
  • Remove some unused code.

@grendello grendello force-pushed the dev/grendel/c++23 branch 2 times, most recently from cc3f7dd to 8187508 Compare November 7, 2024 13:14
@grendello grendello force-pushed the dev/grendel/c++23 branch 5 times, most recently from 88bc1b0 to 614b4aa Compare November 25, 2024 12:19
@grendello grendello marked this pull request as ready for review November 26, 2024 08:42
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.

1 participant