black

sr-lab · Apr 22, 2024 · 18fb1ac · 18fb1ac
1 parent 8e21c24
commit 18fb1ac
Show file tree

Hide file tree

Showing 7 changed files with 41 additions and 17 deletions.
diff --git a/glitch/analysis/design/duplicate_block.py b/glitch/analysis/design/duplicate_block.py
@@ -44,11 +44,13 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
                         if i not in checked:
                             line = self.__get_line(i, lines)
                             error = Error(
-                                "design_duplicate_block", element, element.path, self.code_lines[line - 1]
+                                "design_duplicate_block",
+                                element,
+                                element.path,
+                                self.code_lines[line - 1],
                             )
                             error.line = line
                             errors.append(error)
                             checked.update(range(i, i + 150))
 
-
-        return errors
+        return errors
diff --git a/glitch/analysis/design/improper_alignment.py b/glitch/analysis/design/improper_alignment.py
@@ -11,7 +11,12 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
             errors: List[Error] = []
             for i, line in enumerate(self.code_lines):
                 if "\t" in line:
-                    error = Error("implementation_improper_alignment", element, element.path, repr(element))
+                    error = Error(
+                        "implementation_improper_alignment",
+                        element,
+                        element.path,
+                        repr(element),
+                    )
                     error.line = i + 1
                     errors.append(error)
             return errors

diff --git a/glitch/analysis/design/long_resource.py b/glitch/analysis/design/long_resource.py
@@ -6,7 +6,6 @@
 
 
 class TooManyVariables(DesignSmellChecker):
-
     def check(self, element: CodeElement, file: str) -> List[Error]:
         if isinstance(element, AtomicUnit) and element.type in DesignVisitor.EXEC:
             lines = 0
@@ -18,4 +17,4 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
             if lines > 7:
                 return [Error("design_long_resource", element, file, repr(element))]
 
-        return []
+        return []
diff --git a/glitch/analysis/design/long_statement.py b/glitch/analysis/design/long_statement.py
@@ -11,8 +11,10 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
         if isinstance(element, UnitBlock) and element.type != UnitBlockType.block:
             for i, line in enumerate(self.code_lines):
                 if len(line) > 140:
-                    error = Error("implementation_long_statement", element, element.path, line)
+                    error = Error(
+                        "implementation_long_statement", element, element.path, line
+                    )
                     error.line = i + 1
                     errors.append(error)
-        
-        return errors
+
+        return errors
diff --git a/glitch/analysis/design/multifaceted_abstraction.py b/glitch/analysis/design/multifaceted_abstraction.py
@@ -6,21 +6,27 @@
 
 
 class TooManyVariables(DesignSmellChecker):
-
     def check(self, element: CodeElement, file: str) -> List[Error]:
         if isinstance(element, AtomicUnit) and element.type in DesignVisitor.EXEC:
             if isinstance(element.name, str) and (
                 "&&" in element.name or ";" in element.name or "|" in element.name
             ):
                 return [
-                    Error("design_multifaceted_abstraction", element, file, repr(element))
+                    Error(
+                        "design_multifaceted_abstraction", element, file, repr(element)
+                    )
                 ]
             else:
                 for attribute in element.attributes:
                     value = repr(attribute.value)
                     if "&&" in value or ";" in value or "|" in value:
                         return [
-                            Error("design_multifaceted_abstraction", element, file, repr(element))
+                            Error(
+                                "design_multifaceted_abstraction",
+                                element,
+                                file,
+                                repr(element),
+                            )
                         ]
 
-        return []
+        return []
diff --git a/glitch/analysis/design/too_many_variables.py b/glitch/analysis/design/too_many_variables.py
@@ -21,5 +21,12 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
             if (
                 self.__count_variables(element.variables) / max(len(self.code_lines), 1) > 0.3  # type: ignore
             ):
-                return [Error("implementation_too_many_variables", element, element.path, repr(element))]
-        return []
+                return [
+                    Error(
+                        "implementation_too_many_variables",
+                        element,
+                        element.path,
+                        repr(element),
+                    )
+                ]
+        return []
diff --git a/glitch/analysis/design/unguarded_variable.py b/glitch/analysis/design/unguarded_variable.py
@@ -10,7 +10,10 @@ class UnguardedVariable(DesignSmellChecker):
     def check(self, element: CodeElement, file: str) -> List[Error]:
         errors: List[Error] = []
 
-        if isinstance(element, UnitBlock) and DesignVisitor.VAR_REFER_SYMBOL is not None:
+        if (
+            isinstance(element, UnitBlock)
+            and DesignVisitor.VAR_REFER_SYMBOL is not None
+        ):
             # FIXME could be improved if we considered strings as part of the model
             for i, l in enumerate(self.code_lines):
                 for tuple in re.findall(
@@ -30,4 +33,4 @@ def check(self, element: CodeElement, file: str) -> List[Error]:
                                 error.line = i + 1
                                 errors.append(error)
 
-        return errors
+        return errors