Skip to content

Add header for openPMD-specific errors#1080

Merged
ax3l merged 3 commits intoopenPMD:devfrom
franzpoeschel:error-class
Aug 11, 2021
Merged

Add header for openPMD-specific errors#1080
ax3l merged 3 commits intoopenPMD:devfrom
franzpoeschel:error-class

Conversation

@franzpoeschel
Copy link
Contributor

@franzpoeschel franzpoeschel commented Aug 5, 2021

This is just the scaffolding for the more fine-grained error-handling that we want to implement.
This adds:

  • A class openPMD::Error which should serve as base class for all of openPMD's error types. It should not be instatiated directly, but via child classes.
  • A namespace openPMD::error which bundles all those error types.
  • An example error type openPMD::error::OperationUnsupportedInBackend, without using it.

Related: #983 (this is not a fix, but the fundament for a fix)

TODO:

  • Python?

@ax3l ax3l self-assigned this Aug 11, 2021
@ax3l ax3l self-requested a review August 11, 2021 05:53
Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

Thanks a lot! 🎉

@ax3l ax3l merged commit 5843731 into openPMD:dev Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants