Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2590)

Unified Diff: build/android/pylib/utils/parallelizer.py

Issue 727543003: [Android] Fix new pylint errors in build/android/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/pylib/utils/json_results_generator.py ('k') | build/android/pylib/utils/report_results.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/pylib/utils/parallelizer.py
diff --git a/build/android/pylib/utils/parallelizer.py b/build/android/pylib/utils/parallelizer.py
index 9323c21ad4e2105d135508fcc6978a8e9e357949..9a85b54a5e854a080d91cfa27057d62b79d20031 100644
--- a/build/android/pylib/utils/parallelizer.py
+++ b/build/android/pylib/utils/parallelizer.py
@@ -52,7 +52,7 @@ to parallelize operations that are already fast will incur a net performance
penalty.
"""
-# pylint: disable=W0613
+# pylint: disable=protected-access
from pylib.utils import reraiser_thread
from pylib.utils import watchdog_timer
@@ -194,11 +194,11 @@ class Parallelizer(object):
or '_assertNoShadow'.
"""
if isinstance(self._objs, reraiser_thread.ReraiserThreadGroup):
- assert(not hasattr(self._objs, '_assertNoShadow'))
- assert(not hasattr(self._objs, 'pGet'))
+ assert not hasattr(self._objs, '_assertNoShadow')
+ assert not hasattr(self._objs, attr_name)
else:
- assert(not any(hasattr(o, '_assertNoShadow') for o in self._objs))
- assert(not any(hasattr(o, 'pGet') for o in self._objs))
+ assert not any(hasattr(o, '_assertNoShadow') for o in self._objs)
+ assert not any(hasattr(o, attr_name) for o in self._objs)
class SyncParallelizer(Parallelizer):
« no previous file with comments | « build/android/pylib/utils/json_results_generator.py ('k') | build/android/pylib/utils/report_results.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698