summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitry Jemerov <yole@google.com>2014-07-24 20:33:41 +0200
committerDmitry Jemerov <yole@google.com>2014-07-24 20:33:41 +0200
commit7f2a185aba1a0e5ab796b66230c67af2ff2be380 (patch)
treec42da0a722e0744ebfe564339e771422eac6a3f6
parent103221e61a154aba6be0850a5ff142f3fd216b34 (diff)
downloadpep8-7f2a185aba1a0e5ab796b66230c67af2ff2be380.tar.gz
Fix violations of new rule in pep8.py. #ep14boat
-rwxr-xr-xpep8.py17
1 files changed, 9 insertions, 8 deletions
diff --git a/pep8.py b/pep8.py
index 1dae086..1764005 100755
--- a/pep8.py
+++ b/pep8.py
@@ -506,8 +506,9 @@ def continued_indentation(logical_line, tokens, indent_level, hang_closing,
yield start, "%s continuation line %s" % error
# look for visual indenting
- if (parens[row] and token_type not in (tokenize.NL, tokenize.COMMENT)
- and not indent[depth]):
+ if (parens[row] and
+ token_type not in (tokenize.NL, tokenize.COMMENT) and
+ not indent[depth]):
indent[depth] = start[1]
indent_chances[start[1]] = True
if verbose >= 4:
@@ -943,8 +944,8 @@ def break_around_binary_operator(logical_line, tokens):
# The % character is strictly speaking a binary operator, but the
# common usage seems to be to put it next to the format parameters,
# after a line break.
- return ((token_type == tokenize.OP or text == 'and' or text == 'or')
- and text not in "()[]{},:.;@=%")
+ return ((token_type == tokenize.OP or text in ['and', 'or']) and
+ text not in "()[]{},:.;@=%")
line_break = False
unary_context = True
@@ -954,8 +955,8 @@ def break_around_binary_operator(logical_line, tokens):
if ('\n' in text or '\r' in text) and token_type != tokenize.STRING:
line_break = True
else:
- if (is_binary_operator(token_type, text) and line_break
- and not unary_context):
+ if (is_binary_operator(token_type, text) and line_break and
+ not unary_context):
yield start, "W503 line break before binary operator"
unary_context = text in '([{,;'
line_break = False
@@ -1364,8 +1365,8 @@ class Checker(object):
(start_row, start_col) = start
if prev_row != start_row: # different row
prev_text = self.lines[prev_row - 1][prev_col - 1]
- if prev_text == ',' or (prev_text not in '{[('
- and text not in '}])'):
+ if prev_text == ',' or (prev_text not in '{[(' and
+ text not in '}])'):
text = ' ' + text
elif prev_col != start_col: # different column
text = line[prev_col:start_col] + text