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 signal_handler_*block() #134

Merged
merged 1 commit into from Jul 5, 2016

Conversation

Projects
None yet
3 participants
@Susurrus

Susurrus commented Jul 4, 2016

Closes #133. Tested working correctly with my Gattii project.

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
Member

GuillaumeGomez commented Jul 5, 2016

👍

@gkoz

This comment has been minimized.

Show comment
Hide comment
@gkoz

gkoz Jul 5, 2016

Member

Looks all right. Mind adding signal_handler_disconnect as well?

Member

gkoz commented Jul 5, 2016

Looks all right. Mind adding signal_handler_disconnect as well?

@Susurrus

This comment has been minimized.

Show comment
Hide comment
@Susurrus

Susurrus Jul 5, 2016

I'm not planning on adding _disconnect. It's not necessary for my application and I'd rather focus my efforts there right now.

Susurrus commented Jul 5, 2016

I'm not planning on adding _disconnect. It's not necessary for my application and I'd rather focus my efforts there right now.

@gkoz

This comment has been minimized.

Show comment
Hide comment
@gkoz

gkoz Jul 5, 2016

Member

Thanks for the PR.

Member

gkoz commented Jul 5, 2016

Thanks for the PR.

@gkoz gkoz merged commit 9f77b36 into gtk-rs:master Jul 5, 2016

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