commit eca7d092d42d8b644e30093993c670980c411468
author Andrew Pickford <anpickford@googlemail.com> Tue Jun 26 14:57:37 2018 +0200
committer Andrew Pickford <anpickford@googlemail.com> Wed Jun 27 16:34:50 2018 +0200
tree 787474f836622adb1e04e6142abe04c493773e7b
parent 902c6b6e660f58e2b89c5d8090d0f9e013a060ed

Replace a nested if chain in DictPath with a single if/elif chain

reclass/utils/dictpath.py

diff --git a/reclass/utils/dictpath.py b/reclass/utils/dictpath.py
index 39b9572..57971b4 100644
--- a/reclass/utils/dictpath.py
+++ b/reclass/utils/dictpath.py
@@ -61,17 +61,17 @@
 
     def __init__(self, delim, contents=None):
         self._delim = delim
+
         if contents is None:
             self._parts = []
+        elif isinstance(contents, list):
+            self._parts = contents
+        elif isinstance(contents, six.string_types):
+            self._parts = self._split_string(contents)
+        elif isinstance(contents, tuple):
+            self._parts = list(contents)
         else:
-            if isinstance(contents, list):
-                self._parts = contents
-            elif isinstance(contents, six.string_types):
-                self._parts = self._split_string(contents)
-            elif isinstance(contents, tuple):
-                self._parts = list(contents)
-            else:
-                raise TypeError('DictPath() takes string or list, '\
+            raise TypeError('DictPath() takes string or list, '\
                                 'not %s' % type(contents))
 
     def __repr__(self):