Address code review

Author: corona10

Lib/tempfile.py

@@ -433,8 +433,7 @@ def cleanup(self, windows=(_os.name == 'nt'), unlink=_os.unlink):
             try:
                 if not self.close_called:
                     self.close_called = True
-                    if self.file is not None:
-                        self.file.close()
+                    self.file.close()
             finally:
                 # Windows provides delete-on-close as a primitive, in which
                 # case the file was deleted by self.file.close().
@@ -448,8 +447,7 @@ def close(self):
         if not self.close_called:
             self.close_called = True
             try:
-                if self.file is not None:
-                    self.file.close()
+                self.file.close()
             finally:
                 if self.delete and self.delete_on_close:
                     self.cleanup()

Lib/urllib/error.py

@@ -10,7 +10,7 @@
 an application may want to handle an exception like a regular
 response.
 """
-
+import io
 import urllib.response
 
 __all__ = ['URLError', 'HTTPError', 'ContentTooShortError']
@@ -42,7 +42,10 @@ def __init__(self, url, code, msg, hdrs, fp):
         self.hdrs = hdrs
         self.fp = fp
         self.filename = url
-        self.__super_init(fp, hdrs, url, code)
+        if fp is not None:
+            self.__super_init(fp, hdrs, url, code)
+        else:
+            self.__super_init(io.StringIO(), hdrs, url, code)
 
     def __str__(self):
         return 'HTTP Error %s: %s' % (self.code, self.msg)