Skip to content

Make plugin caller functions noexcept#702

Merged
davidbeckingsale merged 1 commit intodevelopfrom
feature/plugin-noexcept
Dec 5, 2019
Merged

Make plugin caller functions noexcept#702
davidbeckingsale merged 1 commit intodevelopfrom
feature/plugin-noexcept

Conversation

@davidbeckingsale
Copy link
Copy Markdown
Member

Summary

  • This PR is a feature, that makes the two functions that call plugins noexcept

Copy link
Copy Markdown
Member

@trws trws left a comment

Choose a reason for hiding this comment

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

👍
:shipit:

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 4, 2019

Codecov Report

Merging #702 into develop will decrease coverage by 0.001%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##           develop      #702       +/-   ##
=============================================
- Coverage   98.682%   98.681%   -0.002%     
=============================================
  Files           70        70               
  Lines         1290      1289        -1     
=============================================
- Hits          1273      1272        -1     
  Misses          17        17
Impacted Files Coverage Δ
include/RAJA/util/plugins.hpp 100% <100%> (ø) ⬆️
include/RAJA/policy/openmp/scan.hpp 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 408b81f...c12398a. Read the comment docs.

Copy link
Copy Markdown
Member

@rhornung67 rhornung67 left a comment

Choose a reason for hiding this comment

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

Very nice! 🥇

@davidbeckingsale davidbeckingsale merged commit b61a75c into develop Dec 5, 2019
@davidbeckingsale
Copy link
Copy Markdown
Member Author

Just as a note, this will abort if the plugin throws an exception. @trws and I discussed adding a custom error handler - I will leave that for future work 😄

@rhornung67 rhornung67 deleted the feature/plugin-noexcept branch March 26, 2020 17:37
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