PEP-8: extra lines

* Two lines between top-level functions and classes
* One line between methods and properties

Author: acdha

bagit.py

@@ -551,33 +551,40 @@ def _validate_bagittxt(self):
 class BagError(Exception):
     pass
 
+
 class BagValidationError(BagError):
     def __init__(self, message, details=[]):
         self.message = message
         self.details = details
+
     def __str__(self):
         if len(self.details) > 0:
             details = " ; ".join([str(e) for e in self.details])
             return "%s: %s" % (self.message, details)
         return self.message
 
+
 class ManifestErrorDetail(BagError):
     def __init__(self, path):
         self.path = path
 
+
 class ChecksumMismatch(ManifestErrorDetail):
     def __init__(self, path, algorithm=None, expected=None, found=None):
         self.path = path
         self.algorithm = algorithm
         self.expected = expected
         self.found = found
+
     def __str__(self):
         return "%s checksum validation failed (alg=%s expected=%s found=%s)" % (self.path, self.algorithm, self.expected, self.found)
 
+
 class FileMissing(ManifestErrorDetail):
     def __str__(self):
         return "%s exists in manifest but not found on filesystem" % self.path
 
+
 class UnexpectedFile(ManifestErrorDetail):
     def __str__(self):
         return "%s exists on filesystem but is not in manifest" % self.path
@@ -771,6 +778,7 @@ def _walk(data_dir):
                 path = '/'.join(parts)
             yield path
 
+
 def _can_bag(test_dir):
     """returns (unwriteable files/folders)
     """
@@ -780,6 +788,7 @@ def _can_bag(test_dir):
             unwriteable.append(os.path.join(os.path.abspath(test_dir), inode))
     return tuple(unwriteable)
 
+
 def _can_read(test_dir):
     """
     returns ((unreadable_dirs), (unreadable_files))
@@ -795,18 +804,23 @@ def _can_read(test_dir):
                 unreadable_files.append(os.path.join(dirpath, fn))
     return (tuple(unreadable_dirs), tuple(unreadable_files))
 
+
 def _manifest_line_md5(filename):
     return _manifest_line(filename, 'md5')
 
+
 def _manifest_line_sha1(filename):
     return _manifest_line(filename, 'sha1')
 
+
 def _manifest_line_sha256(filename):
     return _manifest_line(filename, 'sha256')
 
+
 def _manifest_line_sha512(filename):
     return _manifest_line(filename, 'sha512')
 
+
 def _hasher(algorithm='md5'):
     if algorithm == 'md5':
         m = hashlib.md5()
@@ -818,6 +832,7 @@ def _hasher(algorithm='md5'):
         m = hashlib.sha512()
     return m
 
+
 def _manifest_line(filename, algorithm='md5'):
     with open(filename, 'rb') as fh:
         m = _hasher(algorithm)
@@ -832,11 +847,13 @@ def _manifest_line(filename, algorithm='md5'):
 
     return (m.hexdigest(), _decode_filename(filename), total_bytes)
 
+
 def _encode_filename(s):
     s = s.replace("\r", "%0D")
     s = s.replace("\n", "%0A")
     return s
 
+
 def _decode_filename(s):
     s = re.sub("%0D", "\r", s, re.IGNORECASE)
     s = re.sub("%0A", "\n", s, re.IGNORECASE)
@@ -850,11 +867,13 @@ def __init__(self, *args, **opts):
         self.bag_info = {}
         optparse.OptionParser.__init__(self, *args, **opts)
 
+
 def _bag_info_store(option, opt, value, parser):
     opt = opt.lstrip('--')
     opt_caps = '-'.join([o.capitalize() for o in opt.split('-')])
     parser.bag_info[opt_caps] = value
 
+
 def _make_opt_parser():
     parser = BagOptionParser(usage='usage: %prog [options] dir1 dir2 ...')
     parser.add_option('--processes', action='store', type="int",
@@ -881,6 +900,7 @@ def _make_opt_parser():
                           action='callback', callback=_bag_info_store)
     return parser
 
+
 def _configure_logging(opts):
     log_format="%(asctime)s - %(levelname)s - %(message)s"
     if opts.quiet: