From: James Addison <jay@jp-hosting.net>
Date: Thu, 4 Feb 2021 17:40:42 +0000 (+0000)
Subject: Clarity: special case: avoid using variables that have the same names as methods
X-Git-Url: https://git.madduck.net/etc/vim.git/commitdiff_plain/89c42a0011721e164738c17a192452ee77d9cc99?ds=sidebyside

Clarity: special case: avoid using variables that have the same names as methods
---

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 "[{("):