[PROTON] Centralize and categorize exception handling#10173
Merged
Conversation
- Changed exception types from std::runtime_error to more specific exceptions such as std::out_of_range, std::invalid_argument, and std::logic_error across multiple files for better error categorization. - Introduced utility functions in Utility/Errors.h to create prefixed error messages for consistency and clarity. - Updated various throw statements to utilize the new error handling functions, improving readability and maintainability of the code. - Ensured that all error messages are prefixed with "[PROTON]" for easier identification in logs.
… InstrumentationProfiler
Jokeren
changed the title
Contributor
Author
|
@CRobeck no major functionality updates. Easy to review |
Contributor
Author
|
Hi @fywkevin can you help stamp the PR? |
CRobeck
approved these changes
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.