Merge pull request #3893 from cgohlke/patch-1

tacaswell · tacaswell · commit c02a515ab924 · 2014-12-11T23:47:06.000-05:00
BUG : Don't close GzipFile before it is used
diff --git a/lib/matplotlib/backends/backend_cairo.py b/lib/matplotlib/backends/backend_cairo.py
@@ -482,17 +482,10 @@ def _save (self, fo, format, **kwargs):
                 raise RuntimeError ('cairo has not been compiled with SVG '
                                     'support enabled')
             if format == 'svgz':
-                filename = fo
                 if is_string_like(fo):
-                    fo = open(fo, 'wb')
-                    close = True
+                    fo = gzip.GzipFile(fo, 'wb')
                 else:
-                    close = False
-                try:
                     fo = gzip.GzipFile(None, 'wb', fileobj=fo)
-                finally:
-                    if close:
-                        fo.close()
             surface = cairo.SVGSurface (fo, width_in_points, height_in_points)
         else:
            warnings.warn ("unknown format: %s" % format)
@@ -530,6 +523,8 @@ def _save (self, fo, format, **kwargs):
 
         ctx.show_page()
         surface.finish()
+        if format == 'svgz':
+            fo.close()
 
 
 FigureCanvas = FigureCanvasCairo
