SF bug 558179.

gvanrossum · gvanrossum · commit d842e07470e9 · 2002-06-05T19:07:39.000Z
Change default for get() back to None.
Will backport to 2.2.1.
diff --git a/Lib/rfc822.py b/Lib/rfc822.py
@@ -425,13 +425,6 @@ def __delitem__(self, name):
         for i in list:
             del self.headers[i]
 
-    def get(self, name, default=""):
-        name = name.lower()
-        if name in self.dict:
-            return self.dict[name]
-        else:
-            return default
-
     def setdefault(self, name, default=""):
         lowername = name.lower()
         if lowername in self.dict:
diff --git a/Lib/test/test_pyclbr.py b/Lib/test/test_pyclbr.py
@@ -105,7 +105,7 @@ def test_easy(self):
                                  '_isfunction',
                                  '_ismodule',
                                  '_classify_class_attrs'])
-        self.checkModule('rfc822')
+        self.checkModule('rfc822', ignore=["get"])
         self.checkModule('difflib')
 
     def test_others(self):
diff --git a/Lib/test/test_rfc822.py b/Lib/test/test_rfc822.py
@@ -18,7 +18,7 @@ def test_get(self):
             'To: "last, first" <userid@foo.net>\n\ntest\n')
         self.assert_(msg.get("to") == '"last, first" <userid@foo.net>')
         self.assert_(msg.get("TO") == '"last, first" <userid@foo.net>')
-        self.assert_(msg.get("No-Such-Header") == "")
+        self.assert_(msg.get("No-Such-Header") is None)
         self.assert_(msg.get("No-Such-Header", "No-Such-Value")
                      == "No-Such-Value")
 

-Original file line number
+Diff line change
             'To: "last, first" <[email protected]>\n\ntest\n')
         self.assert_(msg.get("to") == '"last, first" <[email protected]>')
         self.assert_(msg.get("TO") == '"last, first" <[email protected]>')
 -        self.assert_(msg.get("No-Such-Header") == "")
 +        self.assert_(msg.get("No-Such-Header") is None)
         self.assert_(msg.get("No-Such-Header", "No-Such-Value")
                      == "No-Such-Value")