lint: use generators with all/any helpers
Rather than generate lists before passing to all/any functions, create
a generator and pass that directly. This avoids creating a temporary
list that we then immediately throw away.
BUG=b:231761335
TEST=`./run_tests` passes & CQ passes
Change-Id: Ia22c0118e9a7fadbfa8e59535125d2157761e59b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3633287
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
diff --git a/scripts/cros_sdk.py b/scripts/cros_sdk.py
index 715dca1..caa2d50 100644
--- a/scripts/cros_sdk.py
+++ b/scripts/cros_sdk.py
@@ -162,9 +162,9 @@
ignored_prefixes = [prefix for prefix in all_prefixes if prefix != my_prefix]
for filename in os.listdir(storage_dir):
if (filename == tarball_name or
- any([(filename.startswith(p) and
- not (len(my_prefix) > len(p) and filename.startswith(my_prefix)))
- for p in ignored_prefixes])):
+ any((filename.startswith(p) and
+ not (len(my_prefix) > len(p) and filename.startswith(my_prefix)))
+ for p in ignored_prefixes)):
continue
logging.info('Cleaning up old tarball: %s', filename)
osutils.SafeUnlink(os.path.join(storage_dir, filename))