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

Implement ObjectExt::emit() for emitting arbitrary signals #192

Merged
merged 1 commit into from Jul 6, 2017

Conversation

Projects
None yet
3 participants
@sdroege
Member

sdroege commented Jul 6, 2017

Depends on, and includes, #191

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jul 6, 2017

Member

Before you ask, this is mostly useful for G_SIGNAL_ACTION signals. E.g. GtkWindow's activate-focus signal.

Action signals are basically function calls. They are emitted/called from the outside on the object, and the object handles them internally.

Member

sdroege commented Jul 6, 2017

Before you ask, this is mostly useful for G_SIGNAL_ACTION signals. E.g. GtkWindow's activate-focus signal.

Action signals are basically function calls. They are emitted/called from the outside on the object, and the object handles them internally.

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jul 6, 2017

Member

And updated so that it only includes the relevant commit here now

Member

sdroege commented Jul 6, 2017

And updated so that it only includes the relevant commit here now

@EPashkin

This comment has been minimized.

Show comment
Hide comment
@EPashkin

EPashkin Jul 6, 2017

Member

👍 Thanks again

Member

EPashkin commented Jul 6, 2017

👍 Thanks again

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jul 6, 2017

Member

Thanks! Waiting for CIs.

Member

GuillaumeGomez commented Jul 6, 2017

Thanks! Waiting for CIs.

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jul 6, 2017

Member

CI seems happy :)

Member

sdroege commented Jul 6, 2017

CI seems happy :)

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jul 6, 2017

Member

Indeed. :)

Member

GuillaumeGomez commented Jul 6, 2017

Indeed. :)

@GuillaumeGomez GuillaumeGomez merged commit f874842 into gtk-rs:master Jul 6, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment