Cleanup: pep8

check_source/check_descriptions.py

@@ -116,7 +116,7 @@ def check_duplicates():
     # This is mainly useful for operators,
     # other types have too many false positives
 
-    #for t in (structs, funcs, ops, props):
+    # for t in (structs, funcs, ops, props):
     for t in (ops, ):
         description_dict = {}
         print("")

check_source/check_header_duplicate.py

@@ -87,7 +87,6 @@ def scan_source_recursive(dirpath, is_restore):
         ext = splitext(filename)[1]
         return (ext in {".hpp", ".hxx", ".h", ".hh"})
 
-
     def is_ignore(filename):
         pass
 

check_source/check_spelling.py

@@ -290,8 +290,8 @@ def spell_check_comments(filepath):
 
     for comment in comment_list:
         for w in comment.parse():
-            #if len(w) < 15:
-            #    continue
+            # if len(w) < 15:
+            #     continue
 
             w_lower = w.lower()
             if w_lower in dict_custom or w_lower in dict_ignore:

utils/blend2json.py

@@ -207,7 +207,7 @@ def do_bblock_filter(filters, blend, block, meta_keyval, data_keyval):
                     # ones may still have to be processed...
                 else:
                     block.user_data = min(block.user_data, -rec_iter)
-                    do_break = True # No need to check more filters in exclude case...
+                    do_break = True  # No need to check more filters in exclude case...
                     break
         for k, v in data_keyval:
             if key.search(k) and val.search(v):
@@ -220,11 +220,11 @@ def do_bblock_filter(filters, blend, block, meta_keyval, data_keyval):
                     # ones may still have to be processed...
                 else:
                     block.user_data = min(block.user_data, -rec_iter)
-                    do_break = True # No need to check more filters in exclude case...
+                    do_break = True  # No need to check more filters in exclude case...
                     break
         if include and not has_match:  # Include check failed, implies exclusion.
             block.user_data = min(block.user_data, -rec_iter)
-            do_break = True # No need to check more filters in exclude case...
+            do_break = True  # No need to check more filters in exclude case...
         if do_break:
             break
     # Implicit 'include all' in case no include filter is specified...

utils/credits_git_gen.py

@@ -166,4 +166,3 @@ def main():
 
 if __name__ == "__main__":
     main()
-

utils/git_log_review_commits_advanced.py

@@ -64,6 +64,7 @@ __doc__ = __doc__ + \
              os.path.join(_cwd, ACCEPT_PRETTY_FILE), os.path.join(_cwd, ACCEPT_RELEASELOG_FILE))
 del _cwd
 
+
 class _Getch:
     """
     Gets a single character from standard input.
@@ -636,7 +637,7 @@ def main():
                             c1 = get_cat(ch, len(BUGFIX_CATEGORIES))
                         elif c2 is None:
                             if ch == b'\r':
-                                break;
+                                break
                             elif ch == b'\x7f':  # backspace
                                 c1 = None
                                 continue
@@ -646,7 +647,7 @@ def main():
                         else:
                             print("BUG! this should not happen!")
 
-                    if done_main == False:
+                    if done_main is False:
                         # Go back to main loop, this commit is no more accepted nor rejected.
                         tot_accept -= 1
                         continue