Skip to content

libtxproto: Add free'd callback on tx_io_register_cb#31

Open
rofferom wants to merge 1 commit intocyanreg:masterfrom
rofferom:improve-txioregistercb
Open

libtxproto: Add free'd callback on tx_io_register_cb#31
rofferom wants to merge 1 commit intocyanreg:masterfrom
rofferom:improve-txioregistercb

Conversation

@rofferom
Copy link
Copy Markdown
Contributor

@rofferom rofferom commented Apr 3, 2024

This will allow to free stuff inside the userdata.

This will allow to free stuff inside the userdata.
@cyanreg
Copy link
Copy Markdown
Owner

cyanreg commented Apr 5, 2024

Is there a specific need for this?
Userdata is meant to be some user context that users free themselves, as it is created before registering the callback.

@rofferom
Copy link
Copy Markdown
Contributor Author

rofferom commented Apr 8, 2024

Actually This will allow to free stuff inside the userdata. is a consequence of the patch. At first I wanted to be notified of the AVBufferRef * that is returned. It also allow to use the free callback to understand when the userdata isn't needed anymore.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants