Optimize code

Only calling len(in_row_list) once is better than calling it twice. Co-authored-by: Nick <50747025+mcdonnnj@users.noreply.github.com>
cisagov · Dec 30, 2021 · 84225f5 · 84225f5
1 parent 434d895
commit 84225f5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/mdyml/convert_cisagov.py b/src/mdyml/convert_cisagov.py
@@ -94,11 +94,11 @@ def convert() -> None:
         logging.debug(
             "Processing row %d with %d columns: %s", row_count, len(in_row_list), row
         )
-        if len(in_row_list) != EXPECTED_COLUMN_COUNT:
+        if (row_column_count := len(in_row_list)) != EXPECTED_COLUMN_COUNT:
             logging.warning(
                 "Skipping row %d with unexpected number of columns %d: %s",
                 row_count,
-                len(in_row_list),
+                row_column_count,
                 in_row_list,
             )
             continue