-
-
Notifications
You must be signed in to change notification settings - Fork 72
add Meta classes #74
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 Meta classes #74
Conversation
|
Thanks! Need to have more of a look at this one. I don't like exposing Instead of |
|
For method Buffer.getMeta(final GType api), gst_buffer_get_meta() need a GType. |
|
I have make a try with classes : |
|
Possibly. We might need the caching but not sure we need to expose the Another option here I guess would be to have a So that you have
called like
|
|
Adding a note that this will also require fixing for the GType size issue before merging. |
|
This PR is also lacking some Unit-Tests for the new API. |
|
@MaZderMind yes, the whole thing needs rewriting for post-v1. It's kept open so can discuss or start from scratch on that. |
|
OK, going to close this. Hopefully all functionality here, and more, is covered in #208 . Next step is to add proper wrapper classes for other metas. |
No description provided.