codefactor: fix unnecessary else after return maintainability issue

diff --git a/docs/source/_ext/autodoxy.py b/docs/source/_ext/autodoxy.py
index ea3d805..c82d0fd 100644 (file)
--- a/docs/source/_ext/autodoxy.py
+++ b/docs/source/_ext/autodoxy.py
@@ -333,12 +333,9 @@ class DoxygenClassDocumenter(DoxygenDocumenter):
 
         if want_all:
             return False, ((m.find('name').text, m) for m in all_members)
 
         if want_all:
             return False, ((m.find('name').text, m) for m in all_members)

-        else:
-            if not self.options.members:
-                return False, []
-            else:
-                return False, ((m.find('name').text, m) for m in all_members
-                               if m.find('name').text in self.options.members)
+        if not self.options.members:
+            return False, []
+        return False, ((m.find('name').text, m) for m in all_members if m.find('name').text in self.options.members)

 
     def filter_members(self, members, want_all):
         ret = []
 
     def filter_members(self, members, want_all):
         ret = []

diff --git a/src/simix/simcalls.py b/src/simix/simcalls.py
index 123b4c1..c982a24 100755 (executable)
--- a/src/simix/simcalls.py
+++ b/src/simix/simcalls.py
@@ -178,8 +178,7 @@ class Simcall(object):
                                                                                     self.name,
                                                                                     ''.join(', %s %s' % (arg.rettype(), arg.name)
                                                                                             for i, arg in enumerate(self.args)))
                                                                                     self.name,
                                                                                     ''.join(', %s %s' % (arg.rettype(), arg.name)
                                                                                             for i, arg in enumerate(self.args)))

-        else:
-            return ""
+        return ""

 
 
 def parse(fn):
 
 
 def parse(fn):

diff --git a/tools/tesh/tesh.py b/tools/tesh/tesh.py
index 2e0d94a..ea8f485 100755 (executable)
--- a/tools/tesh/tesh.py
+++ b/tools/tesh/tesh.py
@@ -314,8 +314,8 @@ class Cmd(object):
             vdefault = m.group(2)
             if vname in os.environ:
                 return "$" + vname
             vdefault = m.group(2)
             if vname in os.environ:
                 return "$" + vname

-            else:
-                return vdefault
+            return vdefault
+

         self.args = re.sub(r"\${(\w+):=([^}]*)}", replace_perl_variables, self.args)
 
         # replace bash environment variables ($THINGS) to their values
         self.args = re.sub(r"\${(\w+):=([^}]*)}", replace_perl_variables, self.args)
 
         # replace bash environment variables ($THINGS) to their values