[PROPOSAL] Remove XDR dependency #407
Closed
derobins
started this conversation in
Proposed Change
Replies: 2 comments
-
This will be addressed in 4.3.0 (#423). |
Beta Was this translation helpful? Give feedback.
0 replies
-
Closing since XDR has been hidden in the mfhdf library in 4.3.0 and no symbols are externally visible. The remaining work is not high priority at this time. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
PROBLEM
The venerable XDR library is used to perform I/O in the multi-file API calls. This is library is a problem for several reasons:
PROPOSED FIX
XDR would be removed from HDF4 and replaced with new, significantly smaller, I/O code that provides similar, but more portable, behavior. This would be built directly into the HDF4 library and not as a separate, stand-alone library.
HOW WOULD USERS BE AFFECTED?
PRO
➕ HDF4 would be more portable and easier to build
➕ The HDF4 code would be easier to understand
CON
➖ There are calls in local_nc.h that take a pointer to an XDR struct and these calls (which should probably be in the internal API) would have their signatures change
➖ The next version of HDF4 would not be binary compatible with 4.2.16
NEUTRAL
✔️ All XDR-related configure/CMake options would be removed
✔️ The performance of HDF4 should be unaffected
✔️ The functionality of HDF4 would be unaffected
Beta Was this translation helpful? Give feedback.
All reactions