Cleanup: remove unused / redundant variables from conditionals

This commit is contained in:
James Addison 2021-02-04 18:07:43 +00:00 committed by Łukasz Langa
parent e0d766727d
commit 24700806f6

View File

@ -5971,8 +5971,7 @@ def generate_trailers_to_omit(line: Line, line_length: int) -> Iterator[Set[Leaf
elif leaf.type in CLOSING_BRACKETS: elif leaf.type in CLOSING_BRACKETS:
prev = line.leaves[index - 1] if index > 0 else None prev = line.leaves[index - 1] if index > 0 else None
if ( if (
line.magic_trailing_comma prev
and prev
and prev.type == token.COMMA and prev.type == token.COMMA
and not is_one_tuple_between( and not is_one_tuple_between(
leaf.opening_bracket, leaf, line.leaves leaf.opening_bracket, leaf, line.leaves
@ -5999,8 +5998,7 @@ def generate_trailers_to_omit(line: Line, line_length: int) -> Iterator[Set[Leaf
yield omit yield omit
if ( if (
line.magic_trailing_comma prev
and prev
and prev.type == token.COMMA and prev.type == token.COMMA
and not is_one_tuple_between(leaf.opening_bracket, leaf, line.leaves) and not is_one_tuple_between(leaf.opening_bracket, leaf, line.leaves)
): ):
@ -6659,7 +6657,7 @@ def can_omit_invisible_parens(
# unnecessary. # unnecessary.
return True return True
if line.magic_trailing_comma and penultimate.type == token.COMMA: if penultimate.type == token.COMMA:
# The rightmost non-omitted bracket pair is the one we want to explode on. # The rightmost non-omitted bracket pair is the one we want to explode on.
return True return True