Some cleanup in the Tools directory.

florentx · florentx · commit c20740109d59 · 2012-07-07T17:03:54.000+02:00
diff --git a/Tools/ccbench/ccbench.py b/Tools/ccbench/ccbench.py
@@ -10,7 +10,6 @@
 import time
 import os
 import sys
-import functools
 import itertools
 import threading
 import subprocess
diff --git a/Tools/demo/life.py b/Tools/demo/life.py
@@ -46,38 +46,38 @@ def __init__(self, scr, char=ord('*')):
         self.state = {}
         self.scr = scr
         Y, X = self.scr.getmaxyx()
-        self.X, self.Y = X-2, Y-2-1
+        self.X, self.Y = X - 2, Y - 2 - 1
         self.char = char
         self.scr.clear()
 
         # Draw a border around the board
-        border_line = '+'+(self.X*'-')+'+'
+        border_line = '+' + (self.X * '-') + '+'
         self.scr.addstr(0, 0, border_line)
-        self.scr.addstr(self.Y+1, 0, border_line)
+        self.scr.addstr(self.Y + 1, 0, border_line)
         for y in range(0, self.Y):
-            self.scr.addstr(1+y, 0, '|')
-            self.scr.addstr(1+y, self.X+1, '|')
+            self.scr.addstr(1 + y, 0, '|')
+            self.scr.addstr(1 + y, self.X + 1, '|')
         self.scr.refresh()
 
     def set(self, y, x):
         """Set a cell to the live state"""
-        if x<0 or self.X<=x or y<0 or self.Y<=y:
-            raise ValueError("Coordinates out of range %i,%i"% (y, x))
-        self.state[x,y] = 1
+        if x < 0 or self.X <= x or y < 0 or self.Y <= y:
+            raise ValueError("Coordinates out of range %i,%i" % (y, x))
+        self.state[x, y] = 1
 
     def toggle(self, y, x):
         """Toggle a cell's state between live and dead"""
         if x < 0 or self.X <= x or y < 0 or self.Y <= y:
-            raise ValueError("Coordinates out of range %i,%i"% (y, x))
+            raise ValueError("Coordinates out of range %i,%i" % (y, x))
         if (x, y) in self.state:
             del self.state[x, y]
-            self.scr.addch(y+1, x+1, ' ')
+            self.scr.addch(y + 1, x + 1, ' ')
         else:
             self.state[x, y] = 1
             if curses.has_colors():
                 # Let's pick a random color!
                 self.scr.attrset(curses.color_pair(random.randrange(1, 7)))
-            self.scr.addch(y+1, x+1, self.char)
+            self.scr.addch(y + 1, x + 1, self.char)
             self.scr.attrset(0)
         self.scr.refresh()
 
@@ -88,43 +88,46 @@ def erase(self):
 
     def display(self, update_board=True):
         """Display the whole board, optionally computing one generation"""
-        M,N = self.X, self.Y
+        M, N = self.X, self.Y
         if not update_board:
             for i in range(0, M):
                 for j in range(0, N):
-                    if (i,j) in self.state:
-                        self.scr.addch(j+1, i+1, self.char)
+                    if (i, j) in self.state:
+                        self.scr.addch(j + 1, i + 1, self.char)
                     else:
-                        self.scr.addch(j+1, i+1, ' ')
+                        self.scr.addch(j + 1, i + 1, ' ')
             self.scr.refresh()
             return
 
         d = {}
         self.boring = 1
         for i in range(0, M):
-            L = range( max(0, i-1), min(M, i+2) )
+            L = range(max(0, i - 1), min(M, i + 2))
             for j in range(0, N):
                 s = 0
-                live = (i,j) in self.state
-                for k in range( max(0, j-1), min(N, j+2) ):
+                live = (i, j) in self.state
+                for k in range(max(0, j - 1), min(N, j + 2)):
                     for l in L:
-                        if (l,k) in self.state:
+                        if (l, k) in self.state:
                             s += 1
                 s -= live
                 if s == 3:
                     # Birth
-                    d[i,j] = 1
+                    d[i, j] = 1
                     if curses.has_colors():
                         # Let's pick a random color!
                         self.scr.attrset(curses.color_pair(
                             random.randrange(1, 7)))
-                    self.scr.addch(j+1, i+1, self.char)
+                    self.scr.addch(j + 1, i + 1, self.char)
                     self.scr.attrset(0)
-                    if not live: self.boring = 0
-                elif s == 2 and live: d[i,j] = 1       # Survival
+                    if not live:
+                        self.boring = 0
+                elif s == 2 and live:
+                    # Survival
+                    d[i, j] = 1
                 elif live:
                     # Death
-                    self.scr.addch(j+1, i+1, ' ')
+                    self.scr.addch(j + 1, i + 1, ' ')
                     self.boring = 0
         self.state = d
         self.scr.refresh()
@@ -135,16 +138,17 @@ def make_random(self):
         for i in range(0, self.X):
             for j in range(0, self.Y):
                 if random.random() > 0.5:
-                    self.set(j,i)
+                    self.set(j, i)
 
 
 def erase_menu(stdscr, menu_y):
     "Clear the space where the menu resides"
     stdscr.move(menu_y, 0)
     stdscr.clrtoeol()
-    stdscr.move(menu_y+1, 0)
+    stdscr.move(menu_y + 1, 0)
     stdscr.clrtoeol()
 
+
 def display_menu(stdscr, menu_y):
     "Display the menu of possible keystroke commands"
     erase_menu(stdscr, menu_y)
@@ -154,15 +158,16 @@ def display_menu(stdscr, menu_y):
         stdscr.attrset(curses.color_pair(1))
     stdscr.addstr(menu_y, 4,
         'Use the cursor keys to move, and space or Enter to toggle a cell.')
-    stdscr.addstr(menu_y+1, 4,
+    stdscr.addstr(menu_y + 1, 4,
         'E)rase the board, R)andom fill, S)tep once or C)ontinuously, Q)uit')
     stdscr.attrset(0)
 
+
 def keyloop(stdscr):
     # Clear the screen and display the menu of keys
     stdscr.clear()
     stdscr_y, stdscr_x = stdscr.getmaxyx()
-    menu_y = (stdscr_y-3)-1
+    menu_y = (stdscr_y - 3) - 1
     display_menu(stdscr, menu_y)
 
     # If color, then initialize the color pairs
@@ -179,16 +184,16 @@ def keyloop(stdscr):
     curses.mousemask(curses.BUTTON1_CLICKED)
 
     # Allocate a subwindow for the Life board and create the board object
-    subwin = stdscr.subwin(stdscr_y-3, stdscr_x, 0, 0)
+    subwin = stdscr.subwin(stdscr_y - 3, stdscr_x, 0, 0)
     board = LifeBoard(subwin, char=ord('*'))
     board.display(update_board=False)
 
     # xpos, ypos are the cursor's position
-    xpos, ypos = board.X//2, board.Y//2
+    xpos, ypos = board.X // 2, board.Y // 2
 
     # Main loop:
     while True:
-        stdscr.move(1+ypos, 1+xpos)     # Move the cursor
+        stdscr.move(1 + ypos, 1 + xpos)   # Move the cursor
         c = stdscr.getch()                # Get a keystroke
         if 0 < c < 256:
             c = chr(c)
@@ -224,15 +229,21 @@ def keyloop(stdscr):
                 board.display(update_board=False)
             elif c in 'Ss':
                 board.display()
-            else: pass                  # Ignore incorrect keys
-        elif c == curses.KEY_UP and ypos > 0:            ypos -= 1
-        elif c == curses.KEY_DOWN and ypos < board.Y-1:  ypos += 1
-        elif c == curses.KEY_LEFT and xpos > 0:          xpos -= 1
-        elif c == curses.KEY_RIGHT and xpos < board.X-1: xpos += 1
+            else:
+                # Ignore incorrect keys
+                pass
+        elif c == curses.KEY_UP and ypos > 0:
+            ypos -= 1
+        elif c == curses.KEY_DOWN and ypos + 1 < board.Y:
+            ypos += 1
+        elif c == curses.KEY_LEFT and xpos > 0:
+            xpos -= 1
+        elif c == curses.KEY_RIGHT and xpos + 1 < board.X:
+            xpos += 1
         elif c == curses.KEY_MOUSE:
             mouse_id, mouse_x, mouse_y, mouse_z, button_state = curses.getmouse()
-            if (mouse_x > 0 and mouse_x < board.X+1 and
-                mouse_y > 0 and mouse_y < board.Y+1):
+            if (mouse_x > 0 and mouse_x < board.X + 1 and
+                mouse_y > 0 and mouse_y < board.Y + 1):
                 xpos = mouse_x - 1
                 ypos = mouse_y - 1
                 board.toggle(ypos, xpos)
@@ -245,7 +256,7 @@ def keyloop(stdscr):
 
 
 def main(stdscr):
-    keyloop(stdscr)                    # Enter the main loop
+    keyloop(stdscr)                 # Enter the main loop
 
 if __name__ == '__main__':
     curses.wrapper(main)
diff --git a/Tools/demo/ss1.py b/Tools/demo/ss1.py
@@ -812,7 +812,6 @@ def sync(self):
 
 def test_basic():
     "Basic non-gui self-test."
-    import os
     a = Sheet()
     for x in range(1, 11):
         for y in range(1, 11):
diff --git a/Tools/i18n/pygettext.py b/Tools/i18n/pygettext.py
@@ -163,7 +163,6 @@
 import getopt
 import token
 import tokenize
-import operator
 
 __version__ = '1.5'
 
diff --git a/Tools/iobench/iobench.py b/Tools/iobench/iobench.py
@@ -1,8 +1,6 @@
 # -*- coding: utf-8 -*-
 # This file should be kept compatible with both Python 2.6 and Python >= 3.0.
 
-import functools
-import hashlib
 import itertools
 import os
 import platform
diff --git a/Tools/parser/unparse.py b/Tools/parser/unparse.py
@@ -1,6 +1,5 @@
 "Usage: unparse.py <path to source file>"
 import sys
-import math
 import ast
 import tokenize
 import io
diff --git a/Tools/pybench/pybench.py b/Tools/pybench/pybench.py
@@ -35,7 +35,9 @@
 WITH THE USE OR PERFORMANCE OF THIS SOFTWARE !
 """
 
-import sys, time, operator, platform
+import sys
+import time
+import platform
 from CommandLine import *
 
 try:
@@ -962,8 +964,6 @@ def main(self):
                 bench.name = reportfile
                 pickle.dump(bench,f)
                 f.close()
-            except IOError as reason:
-                print('* Error opening/writing reportfile')
             except IOError as reason:
                 print('* Error opening/writing reportfile %s: %s' % (
                     reportfile,
diff --git a/Tools/unicode/makeunicodedata.py b/Tools/unicode/makeunicodedata.py
@@ -32,7 +32,6 @@
 import zipfile
 
 from textwrap import dedent
-from operator import itemgetter
 
 SCRIPT = sys.argv[0]
 VERSION = "3.2"
