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

Fix Fragment Copy constructor and assignment operator to ensure that #48

Merged
merged 2 commits into from
Mar 5, 2024
Merged

Fix Fragment Copy constructor and assignment operator to ensure that #48

merged 2 commits into from
Mar 5, 2024

Conversation

eflumerf
Copy link
Contributor

@eflumerf eflumerf commented Mar 5, 2024

copies don't get upgraded_header_ raw pointer.

@eflumerf
Fix Fragment Copy constructor and assignment operator to ensure that
d19d159 
copies don't get upgraded_header_ raw pointer.
@eflumerf eflumerf added the bug Something isn't working label Mar 5, 2024
@eflumerf eflumerf requested a review from ron003 March 5, 2024 18:14
@eflumerf eflumerf self-assigned this Mar 5, 2024
ron003
ron003 approved these changes Mar 5, 2024
View reviewed changes
Copy link
Contributor

@ron003 ron003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool

@ron003 ron003 merged commit 7d6acf8 into develop Mar 5, 2024
2 of 4 checks passed
@ron003 ron003 deleted the eflumerf/FixFragmentCopy branch March 5, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ron003 ron003 ron003 approved these changes

Assignees

@eflumerf eflumerf

Labels
bug Something isn't working
Projects
art-daq Work Tracker
Status: 🎉 Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eflumerf @ron003