Refactor error handling code to eliminate internal ID calls
All calls to the H5I routines are now made in API routines (sometimes in FUNC_ENTER/LEAVE_* macros), except for some calls to H5E_clear_stack() within the library, but I'm planning to remove those over time.
Also, made all the library internal error messages into static const variables, instead of malloc'ing them, which means that they can just be referenced and not copied.
Several new and updated auto-generated header files were necessary to enable this.
If we're going to check in the generated header files, we should do that for the other files as well (in a separate PR) and create a GitHub action to complain if they need to be regenerated (in another PR).
Ah, I thought the generated H5E headers were included in the repo. I'll take them out.
Ah, I thought the generated H5E headers were included in the repo. I'll take them out.
Removed
Can I get some reviews on this PR? I'd like to get this change to the H5E package in, so I can work on refactoring the H5E_clear_stack() calls out of the internal library code with this refactor underneath those updates.
My perl isn't exactly strong, but everything looks reasonable to me
Super, thanks!