From 89c42a0011721e164738c17a192452ee77d9cc99 Mon Sep 17 00:00:00 2001 From: James Addison Date: Thu, 4 Feb 2021 17:40:42 +0000 Subject: [PATCH 1/1] Clarity: special case: avoid using variables that have the same names as methods --- src/black/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/black/__init__.py b/src/black/__init__.py index c836b2b..7f727b1 100644 --- a/src/black/__init__.py +++ b/src/black/__init__.py @@ -5008,7 +5008,7 @@ def bracket_split_build_line( result.append(leaf, preformatted=True) for comment_after in original.comments_after(leaf): result.append(comment_after, preformatted=True) - if is_body and should_split(result, opening_bracket): + if is_body and should_split_line(result, opening_bracket): result.should_split = True return result @@ -5813,7 +5813,7 @@ def ensure_visible(leaf: Leaf) -> None: leaf.value = ")" -def should_split(line: Line, opening_bracket: Leaf) -> bool: +def should_split_line(line: Line, opening_bracket: Leaf) -> bool: """Should `line` be immediately split with `delimiter_split()` after RHS?""" if not (opening_bracket.parent and opening_bracket.value in "[{("): -- 2.39.5