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.

Sign up for GitHub

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

Jump to bottom

Create fewer boxes for closures #710

Merged
merged 2 commits into from Jan 28, 2019
Merged

Create fewer boxes for closures #710

merged 2 commits into from Jan 28, 2019

Commits on Jan 28, 2019

  1. Don't needlessly double-box closures for GIO async functions 

    The trampoline takes the concrete type of the closure as type parameter
already and we don't pass around trait objects but boxes of the concrete
type. Double-boxing is unnecessary in this case.
    @sdroege
    sdroege committed Jan 28, 2019
    Copy the full SHA
    7dafae5 View commit details
    Browse the repository at this point in the history

  2. Don't box signal handler closures twice 

    By making the trampolines generic in the concrete type of the closure.
    @sdroege
    sdroege committed Jan 28, 2019
    Copy the full SHA
    752c826 View commit details
    Browse the repository at this point in the history