Skip to content

Commit f16014e

Browse files
author
Alexander Smishlajev
committed
er... straighten the logic.
"if not" with "else" branch is one of the ugliest programming constructs! sorry.
1 parent ded4b0b commit f16014e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

roundup/configuration.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Roundup Issue Tracker configuration support
22
#
3-
# $Id: configuration.py,v 1.3 2004-07-25 13:12:59 a1s Exp $
3+
# $Id: configuration.py,v 1.4 2004-07-25 13:21:38 a1s Exp $
44
#
55
__docformat__ = "restructuredtext"
66

@@ -511,10 +511,10 @@ def __init__(self, tracker_home=None):
511511
_option = _class(self, _section, *_args)
512512
self.add_option(_option)
513513
# load the config if tracker_home given
514-
if tracker_home is not None:
515-
self.load(tracker_home)
516-
else:
514+
if tracker_home is None:
517515
self.init_logging()
516+
else:
517+
self.load(tracker_home)
518518

519519
def add_option(self, option):
520520
"""Adopt a new Option object"""

0 commit comments

Comments
 (0)