[go: up one dir, main page]

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

Regen bindings with new gir {g,s}etter annotations #1759

Merged
merged 2 commits into from
Jun 2, 2024

Conversation

A6GibKm
Copy link
Contributor
@A6GibKm A6GibKm commented Jun 1, 2024

No description provided.

@A6GibKm A6GibKm force-pushed the getter/setter branch 3 times, most recently from 722131c to bb1b6ff Compare June 1, 2024 18:15
@@ -290,11 +295,6 @@ pub trait DisplayExt: IsA<Display> + sealed::Sealed + 'static {
}
}

#[doc(alias = "input-shapes")]
fn is_input_shapes(&self) -> bool {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is gone now and merged with the getter function. Nice!

Copy link
Member
@sdroege sdroege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice to me, some redundant properties got removed (so you probably don't want to backport this @bilelmoussaoui )

@bilelmoussaoui
Copy link
Member

Looks nice to me, some redundant properties got removed (so you probably don't want to backport this @bilelmoussaoui )

yeah, not planning to do any release till the one in mid-july anyways. Will review and merge things once I am done with dealing with the ffi renames in gstreamer

@A6GibKm A6GibKm force-pushed the getter/setter branch 2 times, most recently from 4470b19 to ba86fa6 Compare June 2, 2024 10:14
@bilelmoussaoui
Copy link
Member

Could you update gir & regen?

@A6GibKm
Copy link
Contributor Author
A6GibKm commented Jun 2, 2024

Could you update gir & regen?

Done.

We lost set_paintable and others in the last commit and all these _from_
are rarely used nor idiomatic.
@bilelmoussaoui
Copy link
Member

the typos ci is unrelated, will fix in a separate commit. Thanks!

@bilelmoussaoui bilelmoussaoui merged commit ea55f08 into gtk-rs:master Jun 2, 2024
38 of 40 checks passed
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.

3 participants