NoIssue - Improves converters.py PEP8 compliance

cms/converters.py

 # This file is part of the Adblock Plus web scripts,
 # 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 261 matching lines @@
     include_end_regex = '>'
 
     def resolve_includes(self, text):
         def resolve_include(match):
             name = match.group(1)
             for format_, converter_class in converters.iteritems():
                 if self._params['source'].has_include(name, format_):
                     self._params['includedata'] = (
                         self._params['source'].read_include(name, format))
 
-                    converter = converter_class(self._params, key='includedata'

[Vasily Kuznetsov, 2017/03/09 12:33:27]

This looks kind of awkward with just the closing parentheses on the second line.
Seems that it was better with the second argument on the second line. What do
you think?

[Jon Sonesen, 2017/03/09 14:04:17]

Yep, will do

-                                                )
+                    converter = converter_class(self._params,
+                                                key='includedata')
                     result = converter()
                     self.missing_translations += converter.missing_translations
                     self.total_translations += converter.total_translations
                     return result
             raise Exception('Failed to resolve include {}'
                             'on page {}'.format(name, self._params['page']))
 
         return re.sub(
             r'{}\?\s*include\s+([^\s<>"]+)\s*\?{}'.format(
                 self.include_start_regex,
@@ skipping 183 matching lines @@
                 stack.pop()
             stack[-1]['subitems'].append(item)
             stack.append(item)
         return structured
 
 converters = {
     'html': RawConverter,
     'md': MarkdownConverter,
     'tmpl': TemplateConverter,
 }