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

Remove singleton in config.Config #94

Closed
wants to merge 3 commits into from
Closed

Remove singleton in config.Config #94

wants to merge 3 commits into from

Commits on Sep 2, 2016

  1. Turn config.Config into a normal class 

    The singleton interface is removed to make the class and usage less confusing.
The singleton instance is replaced by a global variable in khard/khard.py.
That should make it clear that modifications to this object are persistent (in
contrast to modification to an object that was apearentrly generated on the
fly and not saved: `Config().set_editor(string)`).  It also makes the point
where the object is actually initialized much clearer.
    @lucc
    lucc committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    c9a6330 View commit details
    Browse the repository at this point in the history

  2. Reduce indent in config.Config by 4 spaces 

    No functional changes and no reformatting were done.
    @lucc
    lucc committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    c3388e5 View commit details
    Browse the repository at this point in the history

  3. Fix line length and line continuations in config.Config

    @lucc
    lucc committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    3c119d3 View commit details
    Browse the repository at this point in the history