Add gsk::RenderReplay bindings manually #2184
Open
+273
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A6GibKm
force-pushed
the
render-replay-bindings
branch
from
16cf7ee to
52c17e6
Compare
gsk4/src/render_replay.rs
Outdated
| impl RenderReplay { | ||
| #[doc(alias = "gsk_render_replay_default")] | ||
| #[allow(clippy::should_implement_trait)] | ||
| pub fn default(&mut self, node: impl AsRef<RenderNode>) -> Option<RenderNode> { |
Member
There was a problem hiding this comment.
I don't think any of those need &mut self
A6GibKm
force-pushed
the
render-replay-bindings
branch
2 times, most recently
from
13e5bfa to
31ed695
Compare
gsk4/src/render_replay.rs
Outdated
| pub fn default(&self, node: impl AsRef<RenderNode>) -> Option<RenderNode> { | ||
| unsafe { | ||
| from_glib_full(ffi::gsk_render_replay_default( | ||
| self.to_glib_none_mut().0, |
Member
There was a problem hiding this comment.
You no longer have access to glib::translate::*, you should do self.ptr.as_ptr() for getters and as_mut_ptr for setters (those would require &mut self)
Contributor
Author
There was a problem hiding this comment.
I don't think https://docs.gtk.org/gsk4/struct.RenderReplay.html has any getter and I am not sure if the non set_ methods should be mut.
Contributor
Author
There was a problem hiding this comment.
I settled for leaving only set_ methods as &mut self.
A6GibKm
force-pushed
the
render-replay-bindings
branch
2 times, most recently
from
006fe2d to
8e972de
Compare
Generated by temporarily adding glib:get-type to the gir file, moving the generated file to src and then - Replace the glib::wrapper macro with `struct` and `impl Drop`. - Replace from_glib_borrow with Self(ptr) - Replace to_glib_none_mut with self.0.as_mut
A6GibKm
force-pushed
the
render-replay-bindings
branch
from
8e972de to
3cec3c9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Generated by temporarily adding glib:get-type to the gir file, moving he
generated file to src and modifying the glib::wrapper macro.
This is based on top of #2183.