From f13eb55d59d80907c9f86574ddd23bce2cb41ff3 Mon Sep 17 00:00:00 2001
From: Raymond Hettinger <python@rcn.com>
Date: Sun, 2 Jun 2002 00:40:05 +0000
Subject: [PATCH] Replace boolean test with is None.

---
 Lib/sgmllib.py     | 2 +-
 Lib/shlex.py       | 8 ++++----
 Lib/shutil.py      | 2 +-
 Lib/smtplib.py     | 2 +-
 Lib/sre_compile.py | 2 +-
 Lib/sre_parse.py   | 4 ++--
 6 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/Lib/sgmllib.py b/Lib/sgmllib.py
index bdca8a60b73..021c8b5cf65 100644
--- a/Lib/sgmllib.py
+++ b/Lib/sgmllib.py
@@ -479,7 +479,7 @@ class TestSGMLParser(SGMLParser):
 def test(args = None):
     import sys
 
-    if not args:
+    if args is None:
         args = sys.argv[1:]
 
     if args and args[0] == '-s':
diff --git a/Lib/shlex.py b/Lib/shlex.py
index e50b9adf904..7ffa79c5440 100644
--- a/Lib/shlex.py
+++ b/Lib/shlex.py
@@ -12,7 +12,7 @@ __all__ = ["shlex"]
 class shlex:
     "A lexical analyzer class for simple shell-like syntaxes."
     def __init__(self, instream=None, infile=None):
-        if instream:
+        if instream is not None:
             self.instream = instream
             self.infile = infile
         else:
@@ -47,7 +47,7 @@ class shlex:
         self.instream = newstream
         self.lineno = 1
         if self.debug:
-            if newfile:
+            if newfile is not None:
                 print 'shlex: pushing to file %s' % (self.infile,)
             else:
                 print 'shlex: pushing to stream %s' % (self.instream,)
@@ -188,9 +188,9 @@ class shlex:
 
     def error_leader(self, infile=None, lineno=None):
         "Emit a C-compiler-like, Emacs-friendly error-message leader."
-        if not infile:
+        if infile is None:
             infile = self.infile
-        if not lineno:
+        if lineno is None:
             lineno = self.lineno
         return "\"%s\", line %d: " % (infile, lineno)
 
diff --git a/Lib/shutil.py b/Lib/shutil.py
index 6e3a27639e5..43401273027 100644
--- a/Lib/shutil.py
+++ b/Lib/shutil.py
@@ -122,7 +122,7 @@ def rmtree(path, ignore_errors=0, onerror=None):
             exc = sys.exc_info()
             if ignore_errors:
                 pass
-            elif onerror:
+            elif onerror is not None:
                 onerror(cmd[0], cmd[1], exc)
             else:
                 raise exc[0], (exc[1][0], exc[1][1] + ' removing '+cmd[1])
diff --git a/Lib/smtplib.py b/Lib/smtplib.py
index 62b19bd5efa..4311d1ab091 100755
--- a/Lib/smtplib.py
+++ b/Lib/smtplib.py
@@ -236,7 +236,7 @@ class SMTP:
             (code, msg) = self.connect(host, port)
             if code != 220:
                 raise SMTPConnectError(code, msg)
-        if local_hostname:
+        if local_hostname is not None:
             self.local_hostname = local_hostname
         else:
             # RFC 2821 says we should use the fqdn in the EHLO/HELO verb, and
diff --git a/Lib/sre_compile.py b/Lib/sre_compile.py
index e48be72285b..1c54f67a49c 100644
--- a/Lib/sre_compile.py
+++ b/Lib/sre_compile.py
@@ -145,7 +145,7 @@ def _compile(code, pattern, flags):
 def _compile_charset(charset, flags, code, fixup=None):
     # compile charset subprogram
     emit = code.append
-    if not fixup:
+    if fixup is None:
         fixup = lambda x: x
     for op, av in _optimize_charset(charset, fixup):
         emit(OPCODES[op])
diff --git a/Lib/sre_parse.py b/Lib/sre_parse.py
index a5a8831319d..1b52967a586 100644
--- a/Lib/sre_parse.py
+++ b/Lib/sre_parse.py
@@ -80,7 +80,7 @@ class Pattern:
     def opengroup(self, name=None):
         gid = self.groups
         self.groups = gid + 1
-        if name:
+        if name is not None:
             ogid = self.groupdict.get(name, None)
             if ogid is not None:
                 raise error, ("redefinition of group name %s as group %d; "
@@ -97,7 +97,7 @@ class SubPattern:
     # a subpattern, in intermediate form
     def __init__(self, pattern, data=None):
         self.pattern = pattern
-        if not data:
+        if data is None:
             data = []
         self.data = data
         self.width = None