Replace boolean test with is None

diff --git a/Lib/formatter.py b/Lib/formatter.py
index 75f4718c20fa7dd4931cf12ad94f67de2d5ce5ff..ccbbdf12659b62cfe6d29709ff2ebf489050d197 100644 (file)
--- a/Lib/formatter.py
+++ b/Lib/formatter.py
@@ -38,7 +38,7 @@ class NullFormatter:
     """
 
     def __init__(self, writer=None):
-        if not writer:
+        if writer is None:
             writer = NullWriter()
         self.writer = writer
     def end_paragraph(self, blankline): pass
@@ -433,7 +433,7 @@ class DumbWriter(NullWriter):
 def test(file = None):
     w = DumbWriter()
     f = AbstractFormatter(w)
-    if file:
+    if file is not None:
         fp = open(file)
     elif sys.argv[1:]:
         fp = open(sys.argv[1])

diff --git a/Lib/ftplib.py b/Lib/ftplib.py
index 21b9e5295c20ad2cd2ccfb744029f2ba26e2e0da..4a3d884eef40e48f5617f3ff41630bbb4a6b96ca 100644 (file)
--- a/Lib/ftplib.py
+++ b/Lib/ftplib.py
@@ -659,7 +659,7 @@ class Netrc:
     __defacct = None
 
     def __init__(self, filename=None):
-        if not filename:
+        if filename is None:
             if os.environ.has_key("HOME"):
                 filename = os.path.join(os.environ["HOME"],
                                         ".netrc")

diff --git a/Lib/gettext.py b/Lib/gettext.py
index 0bea9ed0de198ea19dfbbccf8e6a19753aba084c..253b6d8bc771c4622e027e33acc4aa67fe49e079 100644 (file)
--- a/Lib/gettext.py
+++ b/Lib/gettext.py
@@ -104,7 +104,7 @@ class NullTranslations:
         self._info = {}
         self._charset = None
         self._fallback = None
-        if fp:
+        if fp is not None:
             self._parse(fp)
 
     def _parse(self, fp):

diff --git a/Lib/hmac.py b/Lib/hmac.py
index c9e4ae80478606c3868113301d325dff6f3a0176..ba0a63c780161fcea5ef68fb3987b9d2c979603c 100644 (file)
--- a/Lib/hmac.py
+++ b/Lib/hmac.py
@@ -46,7 +46,7 @@ class HMAC:
         key = key + chr(0) * (blocksize - len(key))
         self.outer.update(_strxor(key, opad))
         self.inner.update(_strxor(key, ipad))
-        if (msg):
+        if msg is not None:
             self.update(msg)
 
 ##    def clear(self):