From 8a68ff96057ec58e524a3e41a2d8dca7b5d016bc Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Mon, 24 Sep 2012 12:15:13 +0900 Subject: 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'pager.py') diff --git a/pager.py b/pager.py index 7086aefa..888b108b 100755 --- a/pager.py +++ b/pager.py @@ -74,11 +74,11 @@ def _BecomePager(pager): # 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]) -- cgit v1.2.3-54-g00ecf