Skip to content

Fix compiler warning on passing uninitialized var as const ref - #6636

Merged
mattjala merged 4 commits into
HDFGroup:developfrom
tbeu:fix-clang-warning
Sep 25, 2026
Merged

mattjala merged 4 commits into
HDFGroup:developfrom
tbeu:fix-clang-warning

Conversation

@tbeu

@tbeu tbeu commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Review Checklist

This PR touches the following areas. Each needs a sign-off
from its listed owners before merging.

✅ All areas have been signed off.

@github-actions
github-actions Bot removed the request for review from vchoi-hdfgroup August 21, 2026 19:41
@hyoklee hyoklee added the Component - C Library Core C library issues (usually in the src directory) label Aug 25, 2026
@hyoklee hyoklee added this to the HDF5 2.2.1 milestone Aug 25, 2026
@hyoklee hyoklee moved this from To be triaged to In progress in HDF5 - TRIAGE & TRACK Aug 25, 2026
@hyoklee

hyoklee commented Aug 25, 2026

Copy link
Copy Markdown
Member

@tbeu, please fix clang format CI failure.

Comment thread src/H5Dint.c Outdated
@nbagha1 nbagha1 modified the milestones: HDF5 2.2.1, HDF5 2.3.0 Sep 11, 2026
@github-actions
github-actions Bot requested a review from mattjala September 24, 2026 15:50
@github-actions

Copy link
Copy Markdown
Contributor

🎉 All checklist items are signed off — @mattjala, this PR is ready to merge.

@github-actions github-actions Bot added the checklist-complete All review checklist areas are signed off label Sep 24, 2026
@mattjala
mattjala merged commit 2854a25 into HDFGroup:develop Sep 25, 2026
129 of 130 checks passed
@tbeu
tbeu deleted the fix-clang-warning branch September 25, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

checklist-complete All review checklist areas are signed off Component - C Library Core C library issues (usually in the src directory)

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants