-
-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hdf5 1 12 #28
Hdf5 1 12 #28
Conversation
github workflow files parser files bin scripts doc corrections
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe that the code in the test/vfd.c file is correct in 1.12 and somehow was mis-merged or lost in develop. That should be fixed, but otherwise, everything else looks OK.
hid_t split_fapl_id = H5I_INVALID_HID; | ||
herr_t ret = SUCCEED; | ||
int ret_value = 0; | ||
H5FD_splitter_vfd_config_t *vfd_config = NULL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here.
|
||
if (splitter_prepare_file_paths(&vfd_config, filename_rw) < 0) { | ||
char * filename_rw = NULL; | ||
H5FD_splitter_vfd_config_t *vfd_config = NULL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, too.
hid_t file_id = H5I_INVALID_HID; | ||
const char * filename_tmp = "splitter_tmp.h5"; | ||
char * filename_rw = NULL; | ||
H5FD_splitter_vfd_config_t *vfd_config = NULL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here
The vfd.c file is identical in 1.10 and develop. splitter test is not 1.8 vfd.c. |
Can you create the issue with any information you have as to where and how the vfd splitter test fails in 1.12? |
…/bounds_compat * commit '181c6110251d7a9e9cc23e14dfd8f52cdb14422b': Correct macro check
The vfd test comments out the splitter test temporarily until it can fixed with later commits.