Use for/else instead of setting 'ok' variable.
authorSybren A. Stüvel <sybren@stuvel.eu>
Mon, 19 Jun 2017 13:25:58 +0000 (15:25 +0200)
committerSybren A. Stüvel <sybren@stuvel.eu>
Mon, 19 Jun 2017 13:28:52 +0000 (15:28 +0200)
This is more efficient, and this use case is exactly what the else clause
is for.

tests/python/bl_load_py_modules.py

index c13679d16f0f5956396de7b7ccabd344b8f5d111..5908ba14718693c8ffb52b34595d736935b402ce 100644 (file)
@@ -211,11 +211,10 @@ def load_modules():
          [(os.sep + f + ".py")  for f in BLACKLIST])
 
     for f in source_files:
-        ok = False
         for ignore in ignore_paths:
             if ignore in f:
-                ok = True
-        if not ok:
+                break
+        else:
             raise Exception("Source file %r not loaded in test" % f)
 
     print("loaded %d modules" % len(loaded_files))