Fix small problems found by PyChecker. (Steve Leblanc)

19 files changed, 11 insertions, 37 deletions

diff --git a/src/CHANGES.txt b/src/CHANGES.txt
index bc885f7..f74cb92 100644
--- a/src/CHANGES.txt
+++ b/src/CHANGES.txt
@@ -56,6 +56,12 @@ RELEASE 0.10 - XXX
   - Add a Clean() method to support removing user-specified targets
     when using the -c option.
 
+  - Add a development script for running SCons through PyChecker.
+
+  - Clean up things found by PyChecker (mostly unnecessary imports).
+
+  - Add a script to use HappyDoc to create HTML class documentation.
+
   From Lachlan O'Dea:
 
   - Make the Environment.get() method return None by default.
diff --git a/src/engine/SCons/Action.py b/src/engine/SCons/Action.py
index d8a08be..61a799c 100644
--- a/src/engine/SCons/Action.py
+++ b/src/engine/SCons/Action.py
@@ -29,13 +29,10 @@ XXX
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import copy
 import os
 import os.path
 import re
 import string
-import sys
-import UserDict
 
 import SCons.Errors
 import SCons.Util
diff --git a/src/engine/SCons/Builder.py b/src/engine/SCons/Builder.py
index 9c91c01..122ee49 100644
--- a/src/engine/SCons/Builder.py
+++ b/src/engine/SCons/Builder.py
@@ -46,8 +46,6 @@ __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 
 import os.path
-import string
-import copy
 from SCons.Errors import UserError
 
 import SCons.Action
diff --git a/src/engine/SCons/Defaults.py b/src/engine/SCons/Defaults.py
index 2016694..bfda9fe 100644
--- a/src/engine/SCons/Defaults.py
+++ b/src/engine/SCons/Defaults.py
@@ -37,9 +37,7 @@ __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 
 import os
-import stat
 import string
-import sys
 import os.path
 
 import SCons.Action
diff --git a/src/engine/SCons/Environment.py b/src/engine/SCons/Environment.py
index 5ebf64c..d731eb3 100644
--- a/src/engine/SCons/Environment.py
+++ b/src/engine/SCons/Environment.py
@@ -34,10 +34,6 @@ import copy
 import os
 import os.path
 import re
-import shutil
-import string
-import sys
-import types
 from UserDict import UserDict
 
 import SCons.Action
diff --git a/src/engine/SCons/Node/FS.py b/src/engine/SCons/Node/FS.py
index 1b627fd..eb23cd8 100644
--- a/src/engine/SCons/Node/FS.py
+++ b/src/engine/SCons/Node/FS.py
@@ -39,8 +39,6 @@ __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 import os
 import os.path
 import string
-import sys
-import types
 from UserDict import UserDict
 
 import SCons.Action
diff --git a/src/engine/SCons/Node/__init__.py b/src/engine/SCons/Node/__init__.py
index bac547c..14f99ed 100644
--- a/src/engine/SCons/Node/__init__.py
+++ b/src/engine/SCons/Node/__init__.py
@@ -47,7 +47,6 @@ __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 
 import copy
-import types
 
 import SCons.Sig
 import SCons.Util
@@ -457,7 +456,7 @@ class Node:
     def get_state(self):
         return self.state
 
-    def current(self):
+    def current(self, calc=None):
         return None
 
     def rfile(self):
diff --git a/src/engine/SCons/Platform/cygwin.py b/src/engine/SCons/Platform/cygwin.py
index fe5a818..bda1c90 100644
--- a/src/engine/SCons/Platform/cygwin.py
+++ b/src/engine/SCons/Platform/cygwin.py
@@ -32,7 +32,6 @@ selection method.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import SCons.Util
 import posix
 
 def generate(env):
diff --git a/src/engine/SCons/Platform/os2.py b/src/engine/SCons/Platform/os2.py
index c389521..631952a 100644
--- a/src/engine/SCons/Platform/os2.py
+++ b/src/engine/SCons/Platform/os2.py
@@ -32,8 +32,6 @@ selection method.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import SCons.Util
-
 def generate(env):
     if not env.has_key('ENV'):
         env['ENV']        = {}
diff --git a/src/engine/SCons/Scanner/C.py b/src/engine/SCons/Scanner/C.py
index b9f2d8f..5a58d12 100644
--- a/src/engine/SCons/Scanner/C.py
+++ b/src/engine/SCons/Scanner/C.py
@@ -30,8 +30,6 @@ This module implements the depenency scanner for C/C++ code.
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 
-import copy
-import os.path
 import re
 
 import SCons.Node
diff --git a/src/engine/SCons/Scanner/Fortran.py b/src/engine/SCons/Scanner/Fortran.py
index 034d5ac..a47e02f 100644
--- a/src/engine/SCons/Scanner/Fortran.py
+++ b/src/engine/SCons/Scanner/Fortran.py
@@ -30,8 +30,6 @@ This module implements the dependency scanner for Fortran code.
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 
-import copy
-import os.path
 import re
 
 import SCons.Node
diff --git a/src/engine/SCons/Scanner/Prog.py b/src/engine/SCons/Scanner/Prog.py
index 081fd5c..b2eb121 100644
--- a/src/engine/SCons/Scanner/Prog.py
+++ b/src/engine/SCons/Scanner/Prog.py
@@ -23,7 +23,6 @@
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import copy
 import string
 
 import SCons.Node
diff --git a/src/engine/SCons/Script/__init__.py b/src/engine/SCons/Script/__init__.py
index ca6536f..a33e373 100644
--- a/src/engine/SCons/Script/__init__.py
+++ b/src/engine/SCons/Script/__init__.py
@@ -44,7 +44,6 @@ import os.path
 import string
 import sys
 import traceback
-import copy
 
 # Strip the script directory from sys.path() so on case-insensitive
 # (WIN32) systems Python doesn't think that the "scons" script is the
diff --git a/src/engine/SCons/Sig/__init__.py b/src/engine/SCons/Sig/__init__.py
index 4fe9423..1fd88a4 100644
--- a/src/engine/SCons/Sig/__init__.py
+++ b/src/engine/SCons/Sig/__init__.py
@@ -29,13 +29,13 @@ The Signature package for the scons software construction utility.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
+import cPickle
 import os
 import os.path
-import string
-import SCons.Node
 import time
+
+import SCons.Node
 import SCons.Warnings
-import cPickle
 
 try:
     import MD5
diff --git a/src/engine/SCons/Taskmaster.py b/src/engine/SCons/Taskmaster.py
index 74703f2..dea0739 100644
--- a/src/engine/SCons/Taskmaster.py
+++ b/src/engine/SCons/Taskmaster.py
@@ -29,7 +29,6 @@ Generic Taskmaster.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import copy
 import string
 import sys
 
diff --git a/src/engine/SCons/Tool/__init__.py b/src/engine/SCons/Tool/__init__.py
index f5cea7e..fbb09af 100644
--- a/src/engine/SCons/Tool/__init__.py
+++ b/src/engine/SCons/Tool/__init__.py
@@ -39,7 +39,6 @@ tool definition.
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
 import imp
-import os
 import sys
 
 import SCons.Errors
diff --git a/src/engine/SCons/Tool/icc.py b/src/engine/SCons/Tool/icc.py
index b21069a..1d64dcb 100644
--- a/src/engine/SCons/Tool/icc.py
+++ b/src/engine/SCons/Tool/icc.py
@@ -33,9 +33,6 @@ selection method.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import os.path
-import string
-
 import SCons.Action
 import SCons.Tool
 import SCons.Errors
diff --git a/src/engine/SCons/Util.py b/src/engine/SCons/Util.py
index 5a1c078..c89e8e6 100644
--- a/src/engine/SCons/Util.py
+++ b/src/engine/SCons/Util.py
@@ -764,6 +764,6 @@ def fs_delete(path, remove=1):
                 if remove: os.rmdir(path)
                 display("Removed directory " + path)
     except OSError, e:
-        print "scons: Could not remove '%s':" % str(t), e.strerror
+        print "scons: Could not remove '%s':" % str(path), e.strerror
 
 display = DisplayEngine()
diff --git a/src/engine/SCons/Warnings.py b/src/engine/SCons/Warnings.py
index bf18ba5..070db26 100644
--- a/src/engine/SCons/Warnings.py
+++ b/src/engine/SCons/Warnings.py
@@ -29,10 +29,6 @@ This file implements the warnings framework for SCons.
 
 __revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
 
-import sys
-import copy
-import traceback
-
 import SCons.Errors
 
 class Warning(SCons.Errors.UserError):