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

Don't lookup the listener so many times #1589

Merged
merged 3 commits into from
Feb 20, 2021
Merged

Don't lookup the listener so many times #1589

merged 3 commits into from
Feb 20, 2021

Commits on Feb 20, 2021

  1. Don't lookup the listener so much 

    A tiny improvement. Found during debugging of #1575. Most notably, the
`provider_exists` is called for a few different providers, and each time
we do the same lookup. This is now avoided by remembering with which
listener we are dealing with.
    @rwols
    rwols committed Feb 20, 2021
    Configuration menu
    Copy the full SHA
    29aad36 View commit details
    Browse the repository at this point in the history

  2. remove unused variable

    @rwols
    rwols committed Feb 20, 2021
    Configuration menu
    Copy the full SHA
    cf94a82 View commit details
    Browse the repository at this point in the history

  3. minor tweak

    @rwols
    rwols committed Feb 20, 2021
    Configuration menu
    Copy the full SHA
    6cd3725 View commit details
    Browse the repository at this point in the history