Skip to content

Commit

Permalink
Avoid recomputing the E266 ignored lines each time
Browse files Browse the repository at this point in the history
  • Loading branch information
PeterJCLaw committed Sep 16, 2022
1 parent 211d822 commit adf1353
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions autopep8.py
Expand Up @@ -834,16 +834,18 @@ def fix_e265(self, result):

self.source[result['line'] - 1] = fixed

def _get_e266_ignored_line_numbers(self):
if not hasattr(self, '_e266_ignored_line_numbers'):
source = ''.join(self.source)
self._e266_ignored_line_numbers = set.union(
multiline_string_lines(source, include_docstrings=True),
commented_out_code_lines(source))
return self._e266_ignored_line_numbers

def fix_e266(self, result):
"""Fix too many block comment hashes."""

source = ''.join(self.source)
# TODO: don't recompute this every time!
ignored_line_numbers = multiline_string_lines(
source,
include_docstrings=True) | set(commented_out_code_lines(source))

if result['line'] in ignored_line_numbers:
if result['line'] in self._get_e266_ignored_line_numbers():
return

# Leave stylistic outlined blocks alone.
Expand Down

0 comments on commit adf1353

Please sign in to comment.