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.

Sign up for GitHub

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

Jump to bottom

[ntuple] rearrange code to avoid a compiler warning #16709

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[ntuple] rearrange code to avoid a compiler warning #16709

wants to merge 1 commit into from
+4 −2

Conversation

silverweed
Copy link
Contributor

Fix a warning in RNTupleSerializer

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

@silverweed silverweed self-assigned this Oct 18, 2024
@github-actions GitHub Actions
Copy link

Test Results

    17 files      17 suites   4d 6h 42m 27s ⏱️
 2 714 tests  2 705 ✅ 0 💤 9 ❌
43 528 runs  43 519 ✅ 0 💤 9 ❌

For more details on these failures, see this check.

Results for commit 09f8b94.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jblomer jblomer jblomer approved these changes

@pcanal pcanal pcanal approved these changes

@hahnjo hahnjo Awaiting requested review from hahnjo

@dpiparo dpiparo Awaiting requested review from dpiparo

@enirolf enirolf Awaiting requested review from enirolf

Assignees

@silverweed silverweed

Labels
in:RNTuple
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@silverweed @jblomer @pcanal