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

add empty gravestone values for Sample, Reply, ReplyError, Query and Hello #1571

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DenisBiryukov91
Copy link
Contributor

@DenisBiryukov91 DenisBiryukov91 commented Oct 29, 2024

add empty gravestone values for Sample, Reply, ReplyError, Query and Hello - required to support move operations in C++

Copy link

PR missing one of the required labels: {'breaking-change', 'new feature', 'dependencies', 'documentation', 'bug', 'enhancement', 'internal'}

@DenisBiryukov91 DenisBiryukov91 added the internal Changes not included in the changelog label Oct 29, 2024
@DenisBiryukov91 DenisBiryukov91 marked this pull request as draft October 29, 2024 17:07
@DenisBiryukov91 DenisBiryukov91 changed the title add empty gravestone values for Sample, Reply, Query and Hello add empty gravestone values for Sample, Reply, ReplyError, Query and Hello Oct 30, 2024
@Mallets
Copy link
Member

Mallets commented Nov 28, 2024

@DenisBiryukov91 is this PR still valid or should we close it?

@DenisBiryukov91
Copy link
Contributor Author

@Mallets. Yes it is still valid, although the related task to provide functionality to move out of loaned types in zenoh-c and make std::move in c++ work more closely to how users would expect, has quite low priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes not included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants