MNT: explicitly pass in file handle to print_figure_impl

tacaswell · kahnchana · commit 6e451f1f22c7 · 2017-03-22T13:23:30.000+05:30
Rather than closing over it and relying on it being defined in the
top-level function namespace.
diff --git a/lib/matplotlib/backends/backend_ps.py b/lib/matplotlib/backends/backend_ps.py
@@ -1069,7 +1069,7 @@ def write(self, *kl, **kwargs):
             creator_str = "matplotlib version " + __version__ + \
                 ", http://matplotlib.org/"
 
-        def print_figure_impl():
+        def print_figure_impl(fh):
             # write the PostScript headers
             if isEPSF:
                 print("%!PS-Adobe-3.0 EPSF-3.0", file=fh)
@@ -1169,7 +1169,7 @@ def print_figure_impl():
             # Write to a temporary file.
             fd, tmpfile = mkstemp()
             with io.open(fd, 'w', encoding='latin-1') as fh:
-                print_figure_impl()
+                print_figure_impl(fh)
         else:
             # Write directly to outfile.
             if passed_in_file_object:
@@ -1187,10 +1187,10 @@ def do_nothing():
                 else:
                     fh = outfile
 
-                print_figure_impl()
+                print_figure_impl(fh)
             else:
                 with io.open(outfile, 'w', encoding='latin-1') as fh:
-                    print_figure_impl()
+                    print_figure_impl(fh)
 
         if rcParams['ps.usedistiller']:
             if rcParams['ps.usedistiller'] == 'ghostscript':
