Noissue - Added flake8-abp and pep8-naming extension and fix reported warnings

abp/filters/renderer.py

 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-2016 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 41 matching lines @@
 
     :returns: a tuple `(lines_iterator, inherited_source)` where
         `inherited_source` is the default source to use for included
         fragments.
     """
     if ':' in name:
         source_name, name_in_source = name.split(':', 1)
         try:
             source = sources[source_name]
         except KeyError:
-            raise IncludeError('Unknown source: \'{}\''.format(source_name),
+            raise IncludeError("Unknown source: '{}'".format(source_name),
                                include_stack)
     else:
         source, name_in_source = default_source, name
 
     if source is None:
-        raise IncludeError('Source name is absent in: \'{}\''.format(name),
+        raise IncludeError("Source name is absent in: '{}'".format(name),
                            include_stack)
 
     return (parse_filterlist(source.get(name_in_source)),
             source if source.is_inheritable else None)
 
 
 def _process_includes(sources, default_source, parent_include_stack, lines):
     """Replace include instructions with the lines of included fragment."""
     for line in lines:
         if line.type == 'include':
@@ skipping 15 matching lines @@
             except (NotFound, ValueError) as exc:
                 raise IncludeError(exc, include_stack)
         else:
             yield line
 
 
 def _process_timestamps(lines):
     """Convert timestamp markers into actual timestamps."""
     for line in lines:
         if line.type == 'comment' and '%timestamp%' in line.text:
-            timestamp = time.strftime("%d %b %Y %H:%M UTC", time.gmtime())
+            timestamp = time.strftime('%d %b %Y %H:%M UTC', time.gmtime())
             yield Comment(text=line.text.replace('%timestamp%', timestamp))
         else:
             yield line
 
 
 def _first_and_rest(iterable):
     """Return the first item from the iterable and the rest as an iterator."""
     iterator = iter(iterable)
     first_item = next(iterator)
     return first_item, iterator
 
 
 def _insert_version(lines):
     """Insert metadata comment with version (a.k.a. date)."""
     first_line, rest = _first_and_rest(lines)
-    version = Metadata('Version', time.strftime("%Y%m%d%H%M", time.gmtime()))
+    version = Metadata('Version', time.strftime('%Y%m%d%H%M', time.gmtime()))
     return itertools.chain([first_line, version], rest)
 
 
 def _remove_duplicates(lines):
     """Remove duplicate metadata and headers."""
     # Always remove checksum -- a checksum coming from a fragment
     # will not match for the rendered list.
     seen = {'Checksum'}
     for i, line in enumerate(lines):
         if line.type == 'metadata':
@@ skipping 13 matching lines @@
     See https://adblockplus.org/filters#special-comments for description
     of the checksum algorithm.
     """
     md5sum = hashlib.md5()
 
     for line in lines:
         if line.type != 'emptyline':
             md5sum.update(line.to_string().encode('utf-8') + b'\n')
         yield line
 
-    sum = base64.b64encode(md5sum.digest()).rstrip(b'=')
-    yield Metadata('Checksum', sum.decode('utf-8'))
+    checksum = base64.b64encode(md5sum.digest()).rstrip(b'=')
+    yield Metadata('Checksum', checksum.decode('utf-8'))
 
 
 def _validate(lines):
     """Validate the final list."""
     first_line, rest = _first_and_rest(lines)
     if first_line.type != 'header':
         raise MissingHeader('No header found at the beginning of the input.')
     return itertools.chain([first_line], rest)
 
 
@@ skipping 10 matching lines @@
     :raises MissingHeader: If the top level fragment doesn't start with a valid
         header.
     """
     _logger.info('Rendering: %s', name)
     lines, default_source = _get_and_parse_fragment(name, sources, top_source)
     lines = _process_includes(sources, default_source, [name], lines)
     for proc in [_process_timestamps, _insert_version, _remove_duplicates,
                  _insert_checksum, _validate]:
         lines = proc(lines)
     return lines