Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Remove remnants of throw() #1799

Merged
merged 1 commit into from
Sep 26, 2022
Merged

Remove remnants of throw() #1799

merged 1 commit into from
Sep 26, 2022

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Sep 21, 2022

the throw() specification has been removed with C++20 and will error out on us.

So rather than that, simply use noexcept, as C++03 is thankfully a thing of the past

Fixes nvbug3799847

the `throw()` specification has been removed with C++20 and will error out on us.

So rather than that, simply use noexcept, as C++03 is thankfully a thing of the past

Fixes nvbug3799847
@miscco
Copy link
Collaborator Author

miscco commented Sep 21, 2022

run tests

Copy link
Collaborator

@gevtushenko gevtushenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup!

@miscco miscco merged commit d3e6fa1 into NVIDIA:main Sep 26, 2022
github-actions bot pushed a commit that referenced this pull request Sep 26, 2022
github-actions bot pushed a commit to clayne/thrust that referenced this pull request Sep 26, 2022
@miscco miscco deleted the no_throw branch September 26, 2022 17:16
github-actions bot pushed a commit to gevtushenko/thrust that referenced this pull request Sep 28, 2022
github-actions bot pushed a commit to mfbalin/thrust that referenced this pull request Oct 2, 2022
alliepiper pushed a commit that referenced this pull request Dec 13, 2022
Remove remnants of `throw()`

(cherry picked from commit d3e6fa1)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants