added __all__ lists to a number of Python modules

added test script and expected output file as well
this closes patch 103297.
__all__ attributes will be added to other modules without first submitting
a patch, just adding the necessary line to the test script to verify
more-or-less correct implementation.

Lib/BaseHTTPServer.py

@@ -63,6 +63,7 @@ XXX To do:
 
 __version__ = "0.2"
 
+__all__ = ["HTTPServer", "BaseHTTPRequestHandler"]
 
 import sys
 import time

Lib/Bastion.py

@@ -26,6 +26,7 @@ bastion is created.
 
 """
 
+__all__ = ["BastionClass", "Bastion"]
 
 from types import MethodType
 

Lib/CGIHTTPServer.py

@@ -19,6 +19,7 @@ SECURITY WARNING: DON'T USE THIS CODE UNLESS YOU ARE INSIDE A FIREWALL
 
 __version__ = "0.4"
 
+__all__ = ["CGIHTTPRequestHandler"]
 
 import os
 import sys

Lib/ConfigParser.py

@@ -89,6 +89,11 @@ import sys
 import string
 import re
 
+__all__ = ["NoSectionError","DuplicateSectionError","NoOptionError",
+           "InterpolationError","InterpolationDepthError","ParsingError",
+           "MissingSectionHeaderError","ConfigParser",
+           "MAX_INTERPOLATION_DEPTH"]
+
 DEFAULTSECT = "DEFAULT"
 
 MAX_INTERPOLATION_DEPTH = 10

Lib/Cookie.py

@@ -228,6 +228,8 @@ try:
 except ImportError:
     raise ImportError, "Cookie.py requires 're' from Python 1.5 or later"
 
+__all__ = ["CookieError","BaseCookie","SimpleCookie","SerialCookie",
+           "SmartCookie","Cookie"]
 
 #
 # Define an exception visible to External modules

Lib/MimeWriter.py

@@ -10,6 +10,7 @@ MimeWriter - the only thing here.
 import string
 import mimetools
 
+__all__ = ["MimeWriter"]
 
 class MimeWriter:
 

Lib/Queue.py

 """A multi-producer, multi-consumer queue."""
 
+__all__ = ["Queue","Empty","Full"]
+
 class Empty(Exception):
     "Exception raised by Queue.get(block=0)/get_nowait()."
     pass

Lib/SimpleHTTPServer.py

@@ -8,6 +8,7 @@ and HEAD requests in a fairly straightforward manner.
 
 __version__ = "0.6"
 
+__all__ = ["SimpleHTTPRequestHandler"]
 
 import os
 import string

Lib/SocketServer.py

@@ -127,6 +127,13 @@ import socket
 import sys
 import os
 
+__all__ = ["TCPServer","UDPServer","ForkingUDPServer","ForkingTCPServer",
+           "ThreadingUDPServer","ThreadingTCPServer","BaseRequestHandler",
+           "StreamRequestHandler","DatagramRequestHandler"]
+if hasattr(socket, "AF_UNIX"):
+    __all__.extend(["UnixStreamServer","UnixDatagramServer",
+                    "ThreadingUnixStreamServer",
+                    "ThreadingUnixDatagramServer"])
 
 class BaseServer:
 

Lib/StringIO.py

@@ -34,6 +34,8 @@ try:
 except ImportError:
     EINVAL = 22
 
+__all__ = ["StringIO"]
+
 EMPTYSTRING = ''
 
 class StringIO:

Lib/UserDict.py

 """A more or less complete user-defined wrapper around dictionary objects."""
 
+__all__ = ["UserDict"]
+
 class UserDict:
     def __init__(self, dict=None):
         self.data = {}

Lib/UserList.py

 """A more or less complete user-defined wrapper around list objects."""
 
+__all__ = ["UserList"]
+
 class UserList:
     def __init__(self, initlist=None):
         self.data = []

Lib/UserString.py

@@ -8,6 +8,8 @@ This module requires Python 1.6 or later.
 from types import StringType, UnicodeType
 import sys
 
+__all__ = ["UserString","MutableString"]
+
 class UserString:
     def __init__(self, seq):
         if isinstance(seq, StringType) or isinstance(seq, UnicodeType):

Lib/aifc.py

@@ -137,6 +137,8 @@ writeframesraw.
 import struct
 import __builtin__
 
+__all__ = ["Error","open","openfp"]
+
 class Error(Exception):
     pass
 

Lib/anydbm.py

@@ -42,6 +42,8 @@ only if it doesn't exist; and 'n' always creates a new database.
 
 """
 
+__all__ = ["error","open"]
+
 try:
     class error(Exception):
         pass

Lib/atexit.py

@@ -5,6 +5,8 @@ upon normal program termination.
 One public function, register, is defined.
 """
 
+__all__ = ["register"]
+
 _exithandlers = []
 def _run_exitfuncs():
     """run any registered exit functions

Lib/audiodev.py

 """Classes for manipulating audio devices (currently only for Sun and SGI)"""
 
+__all__ = ["error","AudioDev"]
+
 class error(Exception):
     pass
 

Lib/base64.py

@@ -6,6 +6,8 @@
 
 import binascii
 
+__all__ = ["encode","decode","encodestring","decodestring"]
+
 MAXLINESIZE = 76 # Excluding the CRLF
 MAXBINSIZE = (MAXLINESIZE/4)*3
 

Lib/bdb.py

@@ -4,6 +4,8 @@ import sys
 import os
 import types
 
+__all__ = ["BdbQuit","Bdb","Breakpoint"]
+
 BdbQuit = 'bdb.BdbQuit' # Exception to give up completely
 
 

Lib/binhex.py

@@ -27,6 +27,8 @@ import struct
 import string
 import binascii
 
+__all__ = ["binhex","hexbin","Error"]
+
 class Error(Exception):
     pass
 

Lib/bisect.py

 """Bisection algorithms."""
 
+__all__ = ["bisect_right","insort_right","bisect_left","insort_left"]
 
 def insort_right(a, x, lo=0, hi=None):
     """Insert item x in list a, and keep it sorted assuming a is sorted.

Lib/calendar.py

@@ -10,6 +10,10 @@ set the first day of the week (0=Monday, 6=Sunday)."""
 # Import functions and variables from time module
 from time import localtime, mktime
 
+__all__ = ["error","setfirstweekday","firstweekday","isleap",
+           "leapdays","weekday","monthrange","monthcalendar",
+           "prmonth","month","prcal","calendar","timegm"]
+
 # Exception raised for bad input (with string parameter for details)
 error = ValueError
 

Lib/cgi.py

@@ -34,6 +34,8 @@ import rfc822
 import UserDict
 from StringIO import StringIO
 
+__all__ = ["MiniFieldStorage","FieldStorage","FormContentDict",
+           "SvFormContentDict","InterpFormContentDict","FormContent"]
 
 # Logging support
 # ===============

Lib/chunk.py

@@ -48,6 +48,8 @@ specifies whether or not chunks are aligned on 2-byte boundaries.  The
 default is 1, i.e. aligned.
 """
 
+__all__ = ["Chunk"]
+           
 class Chunk:
     def __init__(self, file, align = 1, bigendian = 1, inclheader = 0):
         import struct

Lib/cmd.py

@@ -37,6 +37,8 @@ they automatically support Emacs-like command history and editing features.
 
 import string
 
+__all__ = ["Cmd"]
+
 PROMPT = '(Cmd) '
 IDENTCHARS = string.letters + string.digits + '_'
 

Lib/code.py

@@ -10,6 +10,9 @@ import string
 import traceback
 from codeop import compile_command
 
+__all__ = ["InteractiveInterpreter","InteractiveConsole","interact",
+           "compile_command"]
+
 def softspace(file, newvalue):
     oldvalue = 0
     try:

Lib/codecs.py

@@ -17,6 +17,9 @@ except ImportError,why:
     raise SystemError,\
           'Failed to load the builtin codecs: %s' % why
 
+__all__ = ["register","lookup","open","EncodedFile","BOM","BOM_BE",
+           "BOM_LE","BOM32_BE","BOM32_LE","BOM64_BE","BOM64_LE"]
+
 ### Constants
 
 #

Lib/codeop.py

@@ -4,6 +4,8 @@ import sys
 import string
 import traceback
 
+__all__ = ["compile_command"]
+
 def compile_command(source, filename="<input>", symbol="single"):
     r"""Compile a command and determine whether it is incomplete.
 

Lib/colorsys.py

@@ -17,6 +17,8 @@ HSV: Hue, Saturation, Value
 # References:
 # XXX Where's the literature?
 
+__all__ = ["rgb_to_yiq","yiq_to_rgb","rgb_to_hls","hls_to_rgb",
+           "rgb_to_hsv","hsv_to_rgb"]
 
 # Some floating point constants
 

Lib/commands.py

@@ -19,6 +19,8 @@ Encapsulates the basic operation:
  [Note:  it would be nice to add functions to interpret the exit status.]
 """
 
+__all__ = ["getstatusoutput","getoutput","getstatus"]
+
 # Module 'commands'
 #
 # Various tools for executing commands and looking at their output and status.

Lib/compileall.py

@@ -17,6 +17,8 @@ import stat
 import sys
 import py_compile
 
+__all__ = ["compile_dir","compile_path"]
+
 def compile_dir(dir, maxlevels=10, ddir=None, force=0):
     """Byte-compile all modules in the given directory tree.
 

Lib/copy.py

@@ -61,6 +61,8 @@ try:
 except ImportError:
     PyStringMap = None
 
+__all__ = ["Error","error","copy","deepcopy"]
+
 def copy(x):
     """Shallow copy operation on arbitrary Python objects.
 

Lib/copy_reg.py

@@ -6,6 +6,8 @@ C, not for instances of user-defined classes.
 
 from types import ClassType as _ClassType
 
+__all__ = ["pickle","constructor"]
+
 dispatch_table = {}
 safe_constructors = {}
 

Lib/dbhash.py

@@ -2,6 +2,8 @@
 
 import bsddb
 
+__all__ = ["error","open"]
+
 error = bsddb.error                     # Exported for anydbm
 
 def open(file, flag, mode=0666):

Lib/dircache.py

@@ -6,6 +6,8 @@ The annotate() routine appends slashes to directories."""
 
 import os
 
+__all__ = ["listdir","opendir","annotate"]
+
 cache = {}
 
 def listdir(path):

Lib/dis.py

@@ -4,6 +4,10 @@ import sys
 import string
 import types
 
+__all__ = ["dis","disassemble","distb","disco","opname","cmp_op",
+           "hasconst","hasname","hasjrel","hasjabs","haslocal",
+           "hascompare"]
+
 def dis(x=None):
     """Disassemble classes, methods, functions, or code.
 

Lib/robotparser.py

@@ -11,6 +11,8 @@
 """
 import re,string,urlparse,urllib
 
+__all__ = ["RobotFileParser"]
+
 debug = 0
 
 def _debug(msg):

Lib/test/output/test___all__

+test___all__

Lib/test/test___all__.py

+
+from test_support import verify, verbose, TestFailed
+import sys
+
+def check_all(_modname):
+    exec "import %s" % _modname
+    verify(hasattr(sys.modules[_modname],"__all__"),
+           "%s has no __all__ attribute" % _modname)
+    exec "del %s" % _modname
+    exec "from %s import *" % _modname
+    
+    _keys = locals().keys()
+    _keys.remove("_modname")
+    _keys.sort()
+    all = list(sys.modules[_modname].__all__) # in case it's a tuple
+    all.sort()
+    verify(_keys==all,"%s != %s" % (_keys,all))
+
+check_all("BaseHTTPServer")
+check_all("Bastion")
+check_all("CGIHTTPServer")
+check_all("ConfigParser")
+check_all("Cookie")
+check_all("MimeWriter")
+check_all("Queue")
+check_all("SimpleHTTPServer")
+check_all("SocketServer")
+check_all("StringIO")
+check_all("UserDict")
+check_all("UserList")
+check_all("UserString")
+check_all("aifc")
+check_all("anydbm")
+check_all("atexit")
+check_all("audiodev")
+check_all("base64")
+check_all("bdb")
+check_all("binhex")
+check_all("bisect")
+check_all("calendar")
+check_all("cgi")
+check_all("chunk")
+check_all("cmd")
+check_all("code")
+check_all("codecs")
+check_all("codeop")
+check_all("colorsys")
+check_all("commands")
+check_all("compileall")
+check_all("copy")
+check_all("copy_reg")
+check_all("dbhash")
+check_all("dircache")
+check_all("dis")
+check_all("robotparser")