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 functions to get paramspec name quark and type name quark #365

Merged
merged 3 commits into from Jul 30, 2018

Conversation

Projects
None yet
2 participants
@sdroege
Member

sdroege commented Jul 30, 2018

No description provided.

sdroege added some commits Jul 30, 2018

Remove signal::signal_stop_emission()
This is supposed to be used from inside subclasses and we don't bind any
of the API for working with raw signal IDs yet so this is useless in
isolation.
@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jul 30, 2018

Member

Code looks good, I'm restarting CIs.

Member

GuillaumeGomez commented Jul 30, 2018

Code looks good, I'm restarting CIs.

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jul 30, 2018

Member

And I need the Debug impl to be able to use the new quark stuff in GStreamer :)

Member

sdroege commented Jul 30, 2018

And I need the Debug impl to be able to use the new quark stuff in GStreamer :)

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jul 30, 2018

Member

More implementations makes API easier to use so big 👍 for me!

Member

GuillaumeGomez commented Jul 30, 2018

More implementations makes API easier to use so big 👍 for me!

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jul 30, 2018

Member

@GuillaumeGomez all green :)

Member

sdroege commented Jul 30, 2018

@GuillaumeGomez all green :)

@GuillaumeGomez GuillaumeGomez merged commit f704634 into gtk-rs:master Jul 30, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jul 30, 2018

Member

All merged. :)

Member

GuillaumeGomez commented Jul 30, 2018

All merged. :)

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