Merge pull request #3084 from radarhere/health

Health fixes

Tests/test_file_eps.py

@@ -76,7 +76,7 @@ class TestFileEps(PillowTestCase):
         plot_image = Image.open("Tests/images/reqd_showpage.eps")
         target = Image.open("Tests/images/reqd_showpage.png")
 
-        #should not crash/hang
+        # should not crash/hang
         plot_image.load()
         #  fonts could be slightly different
         self.assert_image_similar(plot_image, target, 6)

Tests/test_font_pcf.py

@@ -20,7 +20,7 @@ class TestFontPcf(PillowTestCase):
         with open(fontname, "rb") as test_file:
             font = PcfFontFile.PcfFontFile(test_file)
         self.assertIsInstance(font, FontFile.FontFile)
-        #check the number of characters in the font
+        # check the number of characters in the font
         self.assertEqual(len([_f for _f in font.glyph if _f]), 223)
 
         tempname = self.tempfile("temp.pil")
@@ -66,7 +66,7 @@ class TestFontPcf(PillowTestCase):
     def _test_high_characters(self, message):
         tempname = self.save_font()
         font = ImageFont.load(tempname)
-        im = Image.new("L", (750, 30) , "white")
+        im = Image.new("L", (750, 30), "white")
         draw = ImageDraw.Draw(im)
         draw.text((0, 0), message, "black", font=font)
         with Image.open('Tests/images/high_ascii_chars.png') as target:

Tests/test_image_access.py

@@ -295,7 +295,7 @@ int main(int argc, char* argv[])
         compiler.add_include_dir(sysconfig.get_python_inc())
 
         libdir = sysconfig.get_config_var('LIBDIR') or sysconfig.get_python_inc().replace('include', 'libs')
-        print (libdir)
+        print(libdir)
         compiler.add_library_dir(libdir)
         objects = compiler.compile(['embed_pil.c'])
         compiler.link_executable(objects, 'embed_pil')

Tests/test_image_transform.py

@@ -59,7 +59,7 @@ class TestImageTransform(PillowTestCase):
                                    (0, 0,
                                     w*2, h*2),
                                    Image.BILINEAR,
-                                   fillcolor = 'red')
+                                   fillcolor='red')
 
         self.assertEqual(transformed.getpixel((w-1, h-1)), (255, 0, 0))
 

src/PIL/IcnsImagePlugin.py

@@ -310,8 +310,8 @@ def _save(im, fp, filename):
 
     # create the temporary set of pngs
     iconset = tempfile.mkdtemp('.iconset')
-    provided_images = {im.width:im for im in
+    provided_images = {im.width: im
-                       im.encoderinfo.get("append_images", [])}
+                       for im in im.encoderinfo.get("append_images", [])}
     last_w = None
     for w in [16, 32, 128, 256, 512]:
         prefix = 'icon_{}x{}'.format(w, w)

src/PIL/Image.py

@@ -358,7 +358,7 @@ _initialized = 0
 
 
 def preinit():
-    "Explicitly load standard file format drivers."
+    """Explicitly load standard file format drivers."""
 
     global _initialized
     if _initialized >= 1:
@@ -2246,7 +2246,7 @@ class ImageTransformHandler(object):
 # Debugging
 
 def _wedge():
-    "Create greyscale wedge (for debugging only)"
+    """Create greyscale wedge (for debugging only)"""
 
     return Image()._new(core.wedge("L"))
 

src/PIL/PdfParser.py

@@ -211,7 +211,7 @@ class PdfName:
     def from_pdf_stream(klass, data):
         return klass(PdfParser.interpret_name(data))
 
-    allowed_chars = set(range(33,127)) - set(ord(c) for c in "#%/()<>[]{}")
+    allowed_chars = set(range(33, 127)) - set(ord(c) for c in "#%/()<>[]{}")
 
     def __bytes__(self):
         if str == bytes:  # Python 2.x

src/PIL/PngImagePlugin.py

@@ -663,7 +663,7 @@ _OUTMODES = {
 
 
 def putchunk(fp, cid, *data):
-    "Write a PNG chunk (including CRC field)"
+    """Write a PNG chunk (including CRC field)"""
 
     data = b"".join(data)
 

src/PIL/SpiderImagePlugin.py

@@ -200,7 +200,7 @@ class SpiderImageFile(ImageFile.ImageFile):
 
 # given a list of filenames, return a list of images
 def loadImageSeries(filelist=None):
-    " create a list of Image.images for use in montage "
+    """create a list of Image.images for use in montage"""
     if filelist is None or len(filelist) < 1:
         return
 

src/PIL/features.py

@@ -73,8 +73,8 @@ def get_supported_features():
 
 
 def check(feature):
-    return (feature in modules and check_module(feature) or \
+    return (feature in modules and check_module(feature) or
-            feature in codecs and check_codec(feature) or \
+            feature in codecs and check_codec(feature) or
             feature in features and check_feature(feature))