[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
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
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