From 94eb978a642112b92f79510350441f63f405a80d Mon Sep 17 00:00:00 2001 From: Joshua Harlow Date: Wed, 11 Mar 2015 15:36:29 -0700 Subject: [PATCH] Just use the local conf variable The local conf variable populated by the parent class already ensures the none/falsey case and sets the conf to an empty dictionary (as well as checks if its the expected type and more...); so we should just use that variable instead of duplicating that work. Change-Id: I824263d5253d428f6a803f470e29d99de3dbf42c --- taskflow/persistence/path_based.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/taskflow/persistence/path_based.py b/taskflow/persistence/path_based.py index ea080257..590dc70c 100644 --- a/taskflow/persistence/path_based.py +++ b/taskflow/persistence/path_based.py @@ -36,9 +36,7 @@ class PathBasedBackend(base.Backend): def __init__(self, conf): super(PathBasedBackend, self).__init__(conf) - if conf is None: - conf = {} - self._path = conf.get('path', None) + self._path = self._conf.get('path', None) @property def path(self):