diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-09-24 12:15:13 +0900 |
---|---|---|
committer | Gustaf Lundh <gustaf.lundh@sonymobile.com> | 2012-10-09 12:45:30 +0200 |
commit | 8a68ff96057ec58e524a3e41a2d8dca7b5d016bc (patch) | |
tree | 22f6971e8d3c4a90d11d3704602d073a852328b4 /pager.py | |
parent | e3b1c45aebed329cbc9ad172b1d8e812cf208117 (diff) | |
download | git-repo-8a68ff96057ec58e524a3e41a2d8dca7b5d016bc.tar.gz |
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
Diffstat (limited to 'pager.py')
-rwxr-xr-x | pager.py | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -74,11 +74,11 @@ def _BecomePager(pager): | |||
74 | # ready works around a long-standing bug in popularly | 74 | # ready works around a long-standing bug in popularly |
75 | # available versions of 'less', a better 'more'. | 75 | # available versions of 'less', a better 'more'. |
76 | # | 76 | # |
77 | a, b, c = select.select([0], [], [0]) | 77 | _a, _b, _c = select.select([0], [], [0]) |
78 | 78 | ||
79 | os.environ['LESS'] = 'FRSX' | 79 | os.environ['LESS'] = 'FRSX' |
80 | 80 | ||
81 | try: | 81 | try: |
82 | os.execvp(pager, [pager]) | 82 | os.execvp(pager, [pager]) |
83 | except OSError, e: | 83 | except OSError: |
84 | os.execv('/bin/sh', ['sh', '-c', pager]) | 84 | os.execv('/bin/sh', ['sh', '-c', pager]) |