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

zc_payload_rcinc and zc_attachment_rcinc #238

Closed
milyin opened this issue Jan 25, 2024 · 2 comments
Closed

zc_payload_rcinc and zc_attachment_rcinc #238

milyin opened this issue Jan 25, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@milyin
Copy link
Contributor

milyin commented Jan 25, 2024

Describe the feature

zc_payload_rcinc is available, but no zc_attachment_rcinc is not provided. Therefore, it is impossible for the user to move the attachment outside the callback without copying it. Moreover, when a payload is moved out of the callback and put in a queue with zc_payload_rcinc, when freeing the queue it will panic.

@milyin milyin added the enhancement New feature or request label Jan 25, 2024
@milyin milyin linked a pull request Mar 12, 2024 that will close this issue
@milyin
Copy link
Contributor Author

milyin commented Mar 12, 2024

This will be fixed by #251 : z_owned_sample_t will allow to get ownership of the attachment data and getting it from callback

@milyin
Copy link
Contributor Author

milyin commented May 29, 2024

Owned sample is implemented in dev/1.0.0, attachment now is z_bytes_t, like the playload, no difference between them anymore

@milyin milyin closed this as completed May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant