Skip to content
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

Don't update 'pos' and 'op' fields when using pread/pwrite #4492

Merged
merged 10 commits into from
May 23, 2024

Conversation

qkoziol
Copy link
Contributor

@qkoziol qkoziol commented May 15, 2024

No description provided.

qkoziol and others added 3 commits May 15, 2024 11:53
Instead of reading the absolute minimal possible, use the likely value of
a v2+ superblock w/8-byte addresses & lengths.

Signed-off-by: Quincey Koziol <quincey@koziol.cc>
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
@qkoziol
Copy link
Contributor Author

qkoziol commented May 15, 2024

drat This PR includes the commits from #4491 and should be merged after that one is reveiwed & merged.

@lrknox lrknox added Merge - To 1.14 This needs to be merged to HDF5 1.14 Priority - 2. Medium ⏹ It would be nice to have this in the next release Component - C Library Core C library issues (usually in the src directory) Type - Improvement Improvements that don't add a new feature or functionality labels May 16, 2024
@derobins derobins merged commit fcfab6f into HDFGroup:develop May 23, 2024
58 checks passed
@qkoziol qkoziol deleted the update_sec2_vfd branch May 23, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - C Library Core C library issues (usually in the src directory) Merge - To 1.14 This needs to be merged to HDF5 1.14 Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Improvement Improvements that don't add a new feature or functionality
Projects
Status: Needs Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants