[Request impl] Gracefully error out in ETDump
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 5k
- Forks
- 1.2k
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 581
Description
Context and Target:
In the etdump_flatcc.cpp file, there are instances where the class may directly raise an error and shut down the system when encountering issues. An example can be found here.
The preferred approach is to handle these errors gracefully by returning an error code to the user, allowing them to decide how to address the issue.
TODO:
-
Modify the code to return an error code to the user instead of crashing the system when an error occurs.
If applicable, use theET_CHECK_OR_RETURN_ERRORmacro, as seen here.
Alternatively, manually return an error code, as demonstrated here, if there's extra work likefreeneed to be done. -
Update the return type of the function as necessary and ensure corresponding tests are updated.
Consider breaking this task into two or three pull requests for better structure and clarity.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.