commit 8a68ff96057ec58e524a3e41a2d8dca7b5d016bc
author David Pursehouse <david.pursehouse@sonymobile.com> Mon Sep 24 12:15:13 2012 +0900
committer Gustaf Lundh <gustaf.lundh@sonymobile.com> Tue Oct 09 12:45:30 2012 +0200
tree 22f6971e8d3c4a90d11d3704602d073a852328b4
parent e3b1c45aebed329cbc9ad172b1d8e812cf208117

Coding style cleanup

Fix the following issues reported by pylint:

C0321: More than one statement on a single line
W0622: Redefining built-in 'name'
W0612: Unused variable 'name'
W0613: Unused argument 'name'
W0102: Dangerous default value 'value' as argument
W0105: String statement has no effect

Also fixed a few cases of inconsistent indentation.

Change-Id: Ie0db839e7c57d576cff12d8c055fe87030d00744

pager.py

diff --git a/pager.py b/pager.py
index 7086aef..888b108 100755
--- a/pager.py
+++ b/pager.py
@@ -74,11 +74,11 @@
   # ready works around a long-standing bug in popularly
   # available versions of 'less', a better 'more'.
   #
-  a, b, c = select.select([0], [], [0])
+  _a, _b, _c = select.select([0], [], [0])
 
   os.environ['LESS'] = 'FRSX'
 
   try:
     os.execvp(pager, [pager])
-  except OSError, e:
+  except OSError:
     os.execv('/bin/sh', ['sh', '-c', pager])