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

Payload API #866

Merged
merged 6 commits into from
Mar 27, 2024
Merged

Payload API #866

merged 6 commits into from
Mar 27, 2024

Conversation

Mallets
Copy link
Member

@Mallets Mallets commented Mar 26, 2024

This PR removes the Deref/DerefMut of Payload to ZBuf.
It adds explicit len() and reader() operations.
It also improves some serialization/deserialization.

@eclipse-zenoh-bot
Copy link

@Mallets If this pull request contains a bugfix or a new feature, then please consider using Closes #ISSUE-NUMBER syntax to link it to an issue.

@Mallets Mallets added the enhancement Existing things could work better label Mar 26, 2024
@eclipse-zenoh-bot
Copy link

@Mallets If this pull request contains a bugfix or a new feature, then please consider using Closes #ISSUE-NUMBER syntax to link it to an issue.

@Mallets Mallets merged commit 5a9bf0a into protocol_changes Mar 27, 2024
15 checks passed
@Mallets Mallets deleted the protocol_payload branch March 27, 2024 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@milyin milyin milyin approved these changes

Assignees
No one assigned
Labels
enhancement Existing things could work better
Projects
Zenoh 1.0.0 release
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Mallets @eclipse-zenoh-bot @milyin