diff options
| author | Chris Robinson <[email protected]> | 2023-05-22 02:25:30 -0700 |
|---|---|---|
| committer | Chris Robinson <[email protected]> | 2023-05-22 02:25:30 -0700 |
| commit | eca86489e422f00d96ce59a5c53fa45bdcea593e (patch) | |
| tree | 023fafac55d3dc6888021fccf444f0e0990f3b83 /al/error.cpp | |
| parent | 586b725cc37345601f01473dcc622241ee80d174 (diff) | |
Make the API functions noexcept
Only relevant for C++, but these functions can't throw as it's a C-based API.
Letting the compiler know that helps improve code generation. Extension
callbacks must also not let exceptions leave the callback, or else Bad Things
can happen.
The macro AL_DISABLE_NOEXCEPT may be defined before including the headers to
not mark functions as noexcept, but this should only be done if the caller
can't otherwise be fixed.
Diffstat (limited to 'al/error.cpp')
| -rw-r--r-- | al/error.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/al/error.cpp b/al/error.cpp index 8e6bb3b8..3a1a9ac3 100644 --- a/al/error.cpp +++ b/al/error.cpp @@ -93,7 +93,7 @@ void ALCcontext::setError(ALenum errorCode, const char *msg, ...) /* Special-case alGetError since it (potentially) raises a debug signal and * returns a non-default value for a null context. */ -AL_API ALenum AL_APIENTRY alGetError(void) START_API_FUNC +AL_API ALenum AL_APIENTRY alGetError(void) noexcept { auto context = GetContextRef(); if(!context) UNLIKELY @@ -112,7 +112,7 @@ AL_API ALenum AL_APIENTRY alGetError(void) START_API_FUNC return deferror; } return alGetErrorDirect(context.get()); -} END_API_FUNC +} FORCE_ALIGN ALenum AL_APIENTRY alGetErrorDirect(ALCcontext *context) noexcept { |