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

Unified Diff: build/android/adb_logcat_monitor.py

Issue 153743008: Revert of Enable presubmit pylint in build/android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merging with changes to pylib/linker/test_case.py. Created 6 years, 10 months 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/PRESUBMIT.py ('k') | build/android/adb_logcat_printer.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/adb_logcat_monitor.py
diff --git a/build/android/adb_logcat_monitor.py b/build/android/adb_logcat_monitor.py
index 6e1a68940b0068ae24e8294f90424a8b5cb682fe..35ef7905b24fcb9a30506f5e0d246d3f52085214 100755
--- a/build/android/adb_logcat_monitor.py
+++ b/build/android/adb_logcat_monitor.py
@@ -108,12 +108,12 @@ def main(base_dir, adb_cmd='adb'):
# Set up the alarm for calling 'adb devices'. This is to ensure
# our script doesn't get stuck waiting for a process response
- def TimeoutHandler(_, _):
+ def TimeoutHandler(_, unused_frame):
raise TimeoutException()
signal.signal(signal.SIGALRM, TimeoutHandler)
# Handle SIGTERMs to ensure clean shutdown
- def SigtermHandler(_, _):
+ def SigtermHandler(_, unused_frame):
raise SigtermError()
signal.signal(signal.SIGTERM, SigtermHandler)
« no previous file with comments | « build/android/PRESUBMIT.py ('k') | build/android/adb_logcat_printer.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698