All patches and comments are welcome. Please squash your changes to logical
commits before using git-format-patch and git-send-email to
patches@git.madduck.net.
If you'd read over the Git project's submission guidelines and adhered to them,
I'd be especially grateful.
4 from dataclasses import dataclass, field
18 from black.brackets import COMMA_PRIORITY, DOT_PRIORITY, BracketTracker
19 from black.mode import Mode, Preview
20 from black.nodes import (
29 is_one_sequence_between,
31 is_type_ignore_comment,
32 is_with_or_async_with_stmt,
37 from black.strings import str_width
38 from blib2to3.pgen2 import token
39 from blib2to3.pytree import Leaf, Node
45 LN = Union[Leaf, Node]
50 """Holds leaves and comments. Can be printed with `str(line)`."""
52 mode: Mode = field(repr=False)
54 leaves: List[Leaf] = field(default_factory=list)
55 # keys ordered like `leaves`
56 comments: Dict[LeafID, List[Leaf]] = field(default_factory=dict)
57 bracket_tracker: BracketTracker = field(default_factory=BracketTracker)
58 inside_brackets: bool = False
59 should_split_rhs: bool = False
60 magic_trailing_comma: Optional[Leaf] = None
63 self, leaf: Leaf, preformatted: bool = False, track_bracket: bool = False
65 """Add a new `leaf` to the end of the line.
67 Unless `preformatted` is True, the `leaf` will receive a new consistent
68 whitespace prefix and metadata applied by :class:`BracketTracker`.
69 Trailing commas are maybe removed, unpacked for loop variables are
70 demoted from being delimiters.
72 Inline comments are put aside.
74 has_value = leaf.type in BRACKETS or bool(leaf.value.strip())
78 if token.COLON == leaf.type and self.is_class_paren_empty:
80 if self.leaves and not preformatted:
81 # Note: at this point leaf.prefix should be empty except for
82 # imports, for which we only preserve newlines.
83 leaf.prefix += whitespace(
85 complex_subscript=self.is_complex_subscript(leaf),
88 if self.inside_brackets or not preformatted or track_bracket:
89 self.bracket_tracker.mark(leaf)
90 if self.mode.magic_trailing_comma:
91 if self.has_magic_trailing_comma(leaf):
92 self.magic_trailing_comma = leaf
93 elif self.has_magic_trailing_comma(leaf, ensure_removable=True):
94 self.remove_trailing_comma()
95 if not self.append_comment(leaf):
96 self.leaves.append(leaf)
98 def append_safe(self, leaf: Leaf, preformatted: bool = False) -> None:
99 """Like :func:`append()` but disallow invalid standalone comment structure.
101 Raises ValueError when any `leaf` is appended after a standalone comment
102 or when a standalone comment is not the first leaf on the line.
104 if self.bracket_tracker.depth == 0:
106 raise ValueError("cannot append to standalone comments")
108 if self.leaves and leaf.type == STANDALONE_COMMENT:
110 "cannot append standalone comments to a populated line"
113 self.append(leaf, preformatted=preformatted)
116 def is_comment(self) -> bool:
117 """Is this line a standalone comment?"""
118 return len(self.leaves) == 1 and self.leaves[0].type == STANDALONE_COMMENT
121 def is_decorator(self) -> bool:
122 """Is this line a decorator?"""
123 return bool(self) and self.leaves[0].type == token.AT
126 def is_import(self) -> bool:
127 """Is this an import line?"""
128 return bool(self) and is_import(self.leaves[0])
131 def is_with_or_async_with_stmt(self) -> bool:
132 """Is this a with_stmt line?"""
133 return bool(self) and is_with_or_async_with_stmt(self.leaves[0])
136 def is_class(self) -> bool:
137 """Is this line a class definition?"""
140 and self.leaves[0].type == token.NAME
141 and self.leaves[0].value == "class"
145 def is_stub_class(self) -> bool:
146 """Is this line a class definition with a body consisting only of "..."?"""
147 return self.is_class and self.leaves[-3:] == [
148 Leaf(token.DOT, ".") for _ in range(3)
152 def is_def(self) -> bool:
153 """Is this a function definition? (Also returns True for async defs.)"""
155 first_leaf = self.leaves[0]
160 second_leaf: Optional[Leaf] = self.leaves[1]
163 return (first_leaf.type == token.NAME and first_leaf.value == "def") or (
164 first_leaf.type == token.ASYNC
165 and second_leaf is not None
166 and second_leaf.type == token.NAME
167 and second_leaf.value == "def"
171 def is_stub_def(self) -> bool:
172 """Is this line a function definition with a body consisting only of "..."?"""
173 return self.is_def and self.leaves[-4:] == [Leaf(token.COLON, ":")] + [
174 Leaf(token.DOT, ".") for _ in range(3)
178 def is_class_paren_empty(self) -> bool:
179 """Is this a class with no base classes but using parentheses?
181 Those are unnecessary and should be removed.
185 and len(self.leaves) == 4
187 and self.leaves[2].type == token.LPAR
188 and self.leaves[2].value == "("
189 and self.leaves[3].type == token.RPAR
190 and self.leaves[3].value == ")"
194 def is_triple_quoted_string(self) -> bool:
195 """Is the line a triple quoted string?"""
198 and self.leaves[0].type == token.STRING
199 and self.leaves[0].value.startswith(('"""', "'''"))
203 def opens_block(self) -> bool:
204 """Does this line open a new level of indentation."""
205 if len(self.leaves) == 0:
207 return self.leaves[-1].type == token.COLON
209 def is_fmt_pass_converted(
210 self, *, first_leaf_matches: Optional[Callable[[Leaf], bool]] = None
212 """Is this line converted from fmt off/skip code?
214 If first_leaf_matches is not None, it only returns True if the first
215 leaf of converted code matches.
217 if len(self.leaves) != 1:
219 leaf = self.leaves[0]
221 leaf.type != STANDALONE_COMMENT
222 or leaf.fmt_pass_converted_first_leaf is None
225 return first_leaf_matches is None or first_leaf_matches(
226 leaf.fmt_pass_converted_first_leaf
229 def contains_standalone_comments(self, depth_limit: int = sys.maxsize) -> bool:
230 """If so, needs to be split before emitting."""
231 for leaf in self.leaves:
232 if leaf.type == STANDALONE_COMMENT and leaf.bracket_depth <= depth_limit:
237 def contains_uncollapsable_type_comments(self) -> bool:
240 last_leaf = self.leaves[-1]
241 ignored_ids.add(id(last_leaf))
242 if last_leaf.type == token.COMMA or (
243 last_leaf.type == token.RPAR and not last_leaf.value
245 # When trailing commas or optional parens are inserted by Black for
246 # consistency, comments after the previous last element are not moved
247 # (they don't have to, rendering will still be correct). So we ignore
248 # trailing commas and invisible.
249 last_leaf = self.leaves[-2]
250 ignored_ids.add(id(last_leaf))
254 # A type comment is uncollapsable if it is attached to a leaf
255 # that isn't at the end of the line (since that could cause it
256 # to get associated to a different argument) or if there are
257 # comments before it (since that could cause it to get hidden
260 for leaf_id, comments in self.comments.items():
261 for comment in comments:
262 if is_type_comment(comment):
264 not is_type_ignore_comment(comment)
265 and leaf_id not in ignored_ids
273 def contains_unsplittable_type_ignore(self) -> bool:
277 # If a 'type: ignore' is attached to the end of a line, we
278 # can't split the line, because we can't know which of the
279 # subexpressions the ignore was meant to apply to.
281 # We only want this to apply to actual physical lines from the
282 # original source, though: we don't want the presence of a
283 # 'type: ignore' at the end of a multiline expression to
284 # justify pushing it all onto one line. Thus we
285 # (unfortunately) need to check the actual source lines and
286 # only report an unsplittable 'type: ignore' if this line was
287 # one line in the original code.
289 # Grab the first and last line numbers, skipping generated leaves
290 first_line = next((leaf.lineno for leaf in self.leaves if leaf.lineno != 0), 0)
292 (leaf.lineno for leaf in reversed(self.leaves) if leaf.lineno != 0), 0
295 if first_line == last_line:
296 # We look at the last two leaves since a comma or an
297 # invisible paren could have been added at the end of the
299 for node in self.leaves[-2:]:
300 for comment in self.comments.get(id(node), []):
301 if is_type_ignore_comment(comment):
306 def contains_multiline_strings(self) -> bool:
307 return any(is_multiline_string(leaf) for leaf in self.leaves)
309 def has_magic_trailing_comma(
310 self, closing: Leaf, ensure_removable: bool = False
312 """Return True if we have a magic trailing comma, that is when:
313 - there's a trailing comma here
314 - it's not a one-tuple
315 - it's not a single-element subscript
316 Additionally, if ensure_removable:
317 - it's not from square bracket indexing
318 (specifically, single-element square bracket indexing)
321 closing.type in CLOSING_BRACKETS
323 and self.leaves[-1].type == token.COMMA
327 if closing.type == token.RBRACE:
330 if closing.type == token.RSQB:
333 and closing.parent.type == syms.trailer
334 and closing.opening_bracket
335 and is_one_sequence_between(
336 closing.opening_bracket,
339 brackets=(token.LSQB, token.RSQB),
344 if not ensure_removable:
347 comma = self.leaves[-1]
348 if comma.parent is None:
351 comma.parent.type != syms.subscriptlist
352 or closing.opening_bracket is None
353 or not is_one_sequence_between(
354 closing.opening_bracket,
357 brackets=(token.LSQB, token.RSQB),
364 if closing.opening_bracket is not None and not is_one_sequence_between(
365 closing.opening_bracket, closing, self.leaves
371 def append_comment(self, comment: Leaf) -> bool:
372 """Add an inline or standalone comment to the line."""
374 comment.type == STANDALONE_COMMENT
375 and self.bracket_tracker.any_open_brackets()
380 if comment.type != token.COMMENT:
384 comment.type = STANDALONE_COMMENT
388 last_leaf = self.leaves[-1]
390 last_leaf.type == token.RPAR
391 and not last_leaf.value
393 and len(list(last_leaf.parent.leaves())) <= 3
394 and not is_type_comment(comment)
396 # Comments on an optional parens wrapping a single leaf should belong to
397 # the wrapped node except if it's a type comment. Pinning the comment like
398 # this avoids unstable formatting caused by comment migration.
399 if len(self.leaves) < 2:
400 comment.type = STANDALONE_COMMENT
404 last_leaf = self.leaves[-2]
405 self.comments.setdefault(id(last_leaf), []).append(comment)
408 def comments_after(self, leaf: Leaf) -> List[Leaf]:
409 """Generate comments that should appear directly after `leaf`."""
410 return self.comments.get(id(leaf), [])
412 def remove_trailing_comma(self) -> None:
413 """Remove the trailing comma and moves the comments attached to it."""
414 trailing_comma = self.leaves.pop()
415 trailing_comma_comments = self.comments.pop(id(trailing_comma), [])
416 self.comments.setdefault(id(self.leaves[-1]), []).extend(
417 trailing_comma_comments
420 def is_complex_subscript(self, leaf: Leaf) -> bool:
421 """Return True iff `leaf` is part of a slice with non-trivial exprs."""
422 open_lsqb = self.bracket_tracker.get_open_lsqb()
423 if open_lsqb is None:
426 subscript_start = open_lsqb.next_sibling
428 if isinstance(subscript_start, Node):
429 if subscript_start.type == syms.listmaker:
432 if subscript_start.type == syms.subscriptlist:
433 subscript_start = child_towards(subscript_start, leaf)
434 return subscript_start is not None and any(
435 n.type in TEST_DESCENDANTS for n in subscript_start.pre_order()
438 def enumerate_with_length(
439 self, reversed: bool = False
440 ) -> Iterator[Tuple[Index, Leaf, int]]:
441 """Return an enumeration of leaves with their length.
443 Stops prematurely on multiline strings and standalone comments.
446 Callable[[Sequence[Leaf]], Iterator[Tuple[Index, Leaf]]],
447 enumerate_reversed if reversed else enumerate,
449 for index, leaf in op(self.leaves):
450 length = len(leaf.prefix) + len(leaf.value)
451 if "\n" in leaf.value:
452 return # Multiline strings, we can't continue.
454 for comment in self.comments_after(leaf):
455 length += len(comment.value)
457 yield index, leaf, length
459 def clone(self) -> "Line":
463 inside_brackets=self.inside_brackets,
464 should_split_rhs=self.should_split_rhs,
465 magic_trailing_comma=self.magic_trailing_comma,
468 def __str__(self) -> str:
469 """Render the line."""
473 indent = " " * self.depth
474 leaves = iter(self.leaves)
476 res = f"{first.prefix}{indent}{first.value}"
479 for comment in itertools.chain.from_iterable(self.comments.values()):
484 def __bool__(self) -> bool:
485 """Return True if the line has leaves or comments."""
486 return bool(self.leaves or self.comments)
491 """Intermediate split result from a right hand split."""
496 opening_bracket: Leaf
497 closing_bracket: Leaf
502 """Class that holds information about a block of formatted lines.
504 This is introduced so that the EmptyLineTracker can look behind the standalone
505 comments and adjust their empty lines for class or def lines.
509 previous_block: Optional["LinesBlock"]
512 content_lines: List[str] = field(default_factory=list)
515 def all_lines(self) -> List[str]:
516 empty_line = str(Line(mode=self.mode))
518 [empty_line * self.before] + self.content_lines + [empty_line * self.after]
523 class EmptyLineTracker:
524 """Provides a stateful method that returns the number of potential extra
525 empty lines needed before and after the currently processed line.
527 Note: this tracker works on lines that haven't been split yet. It assumes
528 the prefix of the first leaf consists of optional newlines. Those newlines
529 are consumed by `maybe_empty_lines()` and included in the computation.
533 previous_line: Optional[Line] = None
534 previous_block: Optional[LinesBlock] = None
535 previous_defs: List[Line] = field(default_factory=list)
536 semantic_leading_comment: Optional[LinesBlock] = None
538 def maybe_empty_lines(self, current_line: Line) -> LinesBlock:
539 """Return the number of extra empty lines before and after the `current_line`.
541 This is for separating `def`, `async def` and `class` with extra empty
542 lines (two on module-level).
544 before, after = self._maybe_empty_lines(current_line)
545 previous_after = self.previous_block.after if self.previous_block else 0
547 # Black should not insert empty lines at the beginning
550 if self.previous_line is None
551 else before - previous_after
554 Preview.module_docstring_newlines in current_line.mode
555 and self.previous_block
556 and self.previous_block.previous_block is None
557 and len(self.previous_block.original_line.leaves) == 1
558 and self.previous_block.original_line.is_triple_quoted_string
564 previous_block=self.previous_block,
565 original_line=current_line,
570 # Maintain the semantic_leading_comment state.
571 if current_line.is_comment:
572 if self.previous_line is None or (
573 not self.previous_line.is_decorator
574 # `or before` means this comment already has an empty line before
575 and (not self.previous_line.is_comment or before)
576 and (self.semantic_leading_comment is None or before)
578 self.semantic_leading_comment = block
579 # `or before` means this decorator already has an empty line before
580 elif not current_line.is_decorator or before:
581 self.semantic_leading_comment = None
583 self.previous_line = current_line
584 self.previous_block = block
587 def _maybe_empty_lines(self, current_line: Line) -> Tuple[int, int]:
589 if current_line.depth == 0:
590 max_allowed = 1 if self.mode.is_pyi else 2
591 if current_line.leaves:
592 # Consume the first leaf's extra newlines.
593 first_leaf = current_line.leaves[0]
594 before = first_leaf.prefix.count("\n")
595 before = min(before, max_allowed)
596 first_leaf.prefix = ""
600 user_had_newline = bool(before)
601 depth = current_line.depth
604 while self.previous_defs and self.previous_defs[-1].depth >= depth:
605 previous_def = self.previous_defs.pop()
607 if previous_def is not None:
608 assert self.previous_line is not None
610 if depth and not current_line.is_def and self.previous_line.is_def:
611 # Empty lines between attributes and methods should be preserved.
612 before = 1 if user_had_newline else 0
614 Preview.blank_line_after_nested_stub_class in self.mode
615 and previous_def.is_class
616 and not previous_def.is_stub_class
628 and previous_def.depth
629 and current_line.leaves[-1].type == token.COLON
631 current_line.leaves[0].value
632 not in ("with", "try", "for", "while", "if", "match")
635 # We shouldn't add two newlines between an indented function and
636 # a dependent non-indented clause. This is to avoid issues with
637 # conditional function definitions that are technically top-level
638 # and therefore get two trailing newlines, but look weird and
639 # inconsistent when they're followed by elif, else, etc. This is
640 # worse because these functions only get *one* preceding newline
646 if current_line.is_decorator or current_line.is_def or current_line.is_class:
647 return self._maybe_empty_lines_for_class_or_def(
648 current_line, before, user_had_newline
653 and self.previous_line.is_import
654 and not current_line.is_import
655 and not current_line.is_fmt_pass_converted(first_leaf_matches=is_import)
656 and depth == self.previous_line.depth
658 return (before or 1), 0
662 and self.previous_line.is_class
663 and current_line.is_triple_quoted_string
665 if Preview.no_blank_line_before_class_docstring in current_line.mode:
669 if self.previous_line and self.previous_line.opens_block:
673 def _maybe_empty_lines_for_class_or_def( # noqa: C901
674 self, current_line: Line, before: int, user_had_newline: bool
675 ) -> Tuple[int, int]:
676 if not current_line.is_decorator:
677 self.previous_defs.append(current_line)
678 if self.previous_line is None:
679 # Don't insert empty lines before the first line in the file.
682 if self.previous_line.is_decorator:
683 if self.mode.is_pyi and current_line.is_stub_class:
684 # Insert an empty line after a decorated stub class
689 if self.previous_line.depth < current_line.depth and (
690 self.previous_line.is_class or self.previous_line.is_def
694 comment_to_add_newlines: Optional[LinesBlock] = None
696 self.previous_line.is_comment
697 and self.previous_line.depth == current_line.depth
700 slc = self.semantic_leading_comment
703 and slc.previous_block is not None
704 and not slc.previous_block.original_line.is_class
705 and not slc.previous_block.original_line.opens_block
708 comment_to_add_newlines = slc
713 if current_line.is_class or self.previous_line.is_class:
714 if self.previous_line.depth < current_line.depth:
716 elif self.previous_line.depth > current_line.depth:
718 elif current_line.is_stub_class and self.previous_line.is_stub_class:
719 # No blank line between classes with an empty body
723 # Remove case `self.previous_line.depth > current_line.depth` below when
724 # this becomes stable.
726 # Don't inspect the previous line if it's part of the body of the previous
727 # statement in the same level, we always want a blank line if there's
728 # something with a body preceding.
730 Preview.blank_line_between_nested_and_def_stub_file in current_line.mode
731 and self.previous_line.depth > current_line.depth
735 current_line.is_def or current_line.is_decorator
736 ) and not self.previous_line.is_def:
737 if current_line.depth:
738 # In classes empty lines between attributes and methods should
740 newlines = min(1, before)
742 # Blank line between a block of functions (maybe with preceding
743 # decorators) and a block of non-functions
745 elif self.previous_line.depth > current_line.depth:
750 newlines = 1 if current_line.depth else 2
751 # If a user has left no space after a dummy implementation, don't insert
752 # new lines. This is useful for instance for @overload or Protocols.
754 Preview.dummy_implementations in self.mode
755 and self.previous_line.is_stub_def
756 and not user_had_newline
759 if comment_to_add_newlines is not None:
760 previous_block = comment_to_add_newlines.previous_block
761 if previous_block is not None:
762 comment_to_add_newlines.before = (
763 max(comment_to_add_newlines.before, newlines) - previous_block.after
769 def enumerate_reversed(sequence: Sequence[T]) -> Iterator[Tuple[Index, T]]:
770 """Like `reversed(enumerate(sequence))` if that were possible."""
771 index = len(sequence) - 1
772 for element in reversed(sequence):
773 yield (index, element)
778 new_line: Line, old_line: Line, leaves: List[Leaf], preformatted: bool = False
781 Append leaves (taken from @old_line) to @new_line, making sure to fix the
782 underlying Node structure where appropriate.
784 All of the leaves in @leaves are duplicated. The duplicates are then
785 appended to @new_line and used to replace their originals in the underlying
786 Node structure. Any comments attached to the old leaves are reattached to
790 set(@leaves) is a subset of set(@old_line.leaves).
792 for old_leaf in leaves:
793 new_leaf = Leaf(old_leaf.type, old_leaf.value)
794 replace_child(old_leaf, new_leaf)
795 new_line.append(new_leaf, preformatted=preformatted)
797 for comment_leaf in old_line.comments_after(old_leaf):
798 new_line.append(comment_leaf, preformatted=True)
801 def is_line_short_enough( # noqa: C901
802 line: Line, *, mode: Mode, line_str: str = ""
804 """For non-multiline strings, return True if `line` is no longer than `line_length`.
805 For multiline strings, looks at the context around `line` to determine
806 if it should be inlined or split up.
807 Uses the provided `line_str` rendering, if any, otherwise computes a new one.
810 line_str = line_to_string(line)
812 width = str_width if mode.preview else len
814 if Preview.multiline_string_handling not in mode:
816 width(line_str) <= mode.line_length
817 and "\n" not in line_str # multiline strings
818 and not line.contains_standalone_comments()
821 if line.contains_standalone_comments():
823 if "\n" not in line_str:
824 # No multiline strings (MLS) present
825 return width(line_str) <= mode.line_length
827 first, *_, last = line_str.split("\n")
828 if width(first) > mode.line_length or width(last) > mode.line_length:
831 # Traverse the AST to examine the context of the multiline string (MLS),
832 # tracking aspects such as depth and comma existence,
833 # to determine whether to split the MLS or keep it together.
834 # Depth (which is based on the existing bracket_depth concept)
835 # is needed to determine nesting level of the MLS.
836 # Includes special case for trailing commas.
837 commas: List[int] = [] # tracks number of commas per depth level
838 multiline_string: Optional[Leaf] = None
839 # store the leaves that contain parts of the MLS
840 multiline_string_contexts: List[LN] = []
842 max_level_to_update: Union[int, float] = math.inf # track the depth of the MLS
843 for i, leaf in enumerate(line.leaves):
844 if max_level_to_update == math.inf:
845 had_comma: Optional[int] = None
846 if leaf.bracket_depth + 1 > len(commas):
848 elif leaf.bracket_depth + 1 < len(commas):
849 had_comma = commas.pop()
851 had_comma is not None
852 and multiline_string is not None
853 and multiline_string.bracket_depth == leaf.bracket_depth + 1
855 # Have left the level with the MLS, stop tracking commas
856 max_level_to_update = leaf.bracket_depth
858 # MLS was in parens with at least one comma - force split
861 if leaf.bracket_depth <= max_level_to_update and leaf.type == token.COMMA:
862 # Ignore non-nested trailing comma
863 # directly after MLS/MLS-containing expression
864 ignore_ctxs: List[Optional[LN]] = [None]
865 ignore_ctxs += multiline_string_contexts
866 if not (leaf.prev_sibling in ignore_ctxs and i == len(line.leaves) - 1):
867 commas[leaf.bracket_depth] += 1
868 if max_level_to_update != math.inf:
869 max_level_to_update = min(max_level_to_update, leaf.bracket_depth)
871 if is_multiline_string(leaf):
872 if len(multiline_string_contexts) > 0:
873 # >1 multiline string cannot fit on a single line - force split
875 multiline_string = leaf
877 # fetch the leaf components of the MLS in the AST
878 while str(ctx) in line_str:
879 multiline_string_contexts.append(ctx)
880 if ctx.parent is None:
884 # May not have a triple-quoted multiline string at all,
885 # in case of a regular string with embedded newlines and line continuations
886 if len(multiline_string_contexts) == 0:
889 return all(val == 0 for val in commas)
892 def can_be_split(line: Line) -> bool:
893 """Return False if the line cannot be split *for sure*.
895 This is not an exhaustive search but a cheap heuristic that we can use to
896 avoid some unfortunate formattings (mostly around wrapping unsplittable code
897 in unnecessary parentheses).
903 if leaves[0].type == token.STRING and leaves[1].type == token.DOT:
907 for leaf in leaves[-2::-1]:
908 if leaf.type in OPENING_BRACKETS:
909 if next.type not in CLOSING_BRACKETS:
913 elif leaf.type == token.DOT:
915 elif leaf.type == token.NAME:
916 if not (next.type == token.DOT or next.type in OPENING_BRACKETS):
919 elif leaf.type not in CLOSING_BRACKETS:
922 if dot_count > 1 and call_count > 1:
928 def can_omit_invisible_parens(
932 """Does `rhs.body` have a shape safe to reformat without optional parens around it?
934 Returns True for only a subset of potentially nice looking formattings but
935 the point is to not return false positives that end up producing lines that
939 bt = line.bracket_tracker
940 if not bt.delimiters:
941 # Without delimiters the optional parentheses are useless.
944 max_priority = bt.max_delimiter_priority()
945 delimiter_count = bt.delimiter_count_with_priority(max_priority)
946 if delimiter_count > 1:
947 # With more than one delimiter of a kind the optional parentheses read better.
950 if delimiter_count == 1:
952 Preview.wrap_multiple_context_managers_in_parens in line.mode
953 and max_priority == COMMA_PRIORITY
954 and rhs.head.is_with_or_async_with_stmt
956 # For two context manager with statements, the optional parentheses read
957 # better. In this case, `rhs.body` is the context managers part of
958 # the with statement. `rhs.head` is the `with (` part on the previous
961 # Otherwise it may also read better, but we don't do it today and requires
962 # careful considerations for all possible cases. See
963 # https://github.com/psf/black/issues/2156.
965 if max_priority == DOT_PRIORITY:
966 # A single stranded method call doesn't require optional parentheses.
969 assert len(line.leaves) >= 2, "Stranded delimiter"
971 # With a single delimiter, omit if the expression starts or ends with
973 first = line.leaves[0]
974 second = line.leaves[1]
975 if first.type in OPENING_BRACKETS and second.type not in CLOSING_BRACKETS:
976 if _can_omit_opening_paren(line, first=first, line_length=line_length):
979 # Note: we are not returning False here because a line might have *both*
980 # a leading opening bracket and a trailing closing bracket. If the
981 # opening bracket doesn't match our rule, maybe the closing will.
983 penultimate = line.leaves[-2]
984 last = line.leaves[-1]
987 last.type == token.RPAR
988 or last.type == token.RBRACE
990 # don't use indexing for omitting optional parentheses;
992 last.type == token.RSQB
994 and last.parent.type != syms.trailer
997 if penultimate.type in OPENING_BRACKETS:
998 # Empty brackets don't help.
1001 if is_multiline_string(first):
1002 # Additional wrapping of a multiline string in this situation is
1006 if _can_omit_closing_paren(line, last=last, line_length=line_length):
1012 def _can_omit_opening_paren(line: Line, *, first: Leaf, line_length: int) -> bool:
1013 """See `can_omit_invisible_parens`."""
1015 length = 4 * line.depth
1017 for _index, leaf, leaf_length in line.enumerate_with_length():
1018 if leaf.type in CLOSING_BRACKETS and leaf.opening_bracket is first:
1021 length += leaf_length
1022 if length > line_length:
1025 if leaf.type in OPENING_BRACKETS:
1026 # There are brackets we can further split on.
1030 # checked the entire string and line length wasn't exceeded
1031 if len(line.leaves) == _index + 1:
1037 def _can_omit_closing_paren(line: Line, *, last: Leaf, line_length: int) -> bool:
1038 """See `can_omit_invisible_parens`."""
1039 length = 4 * line.depth
1040 seen_other_brackets = False
1041 for _index, leaf, leaf_length in line.enumerate_with_length():
1042 length += leaf_length
1043 if leaf is last.opening_bracket:
1044 if seen_other_brackets or length <= line_length:
1047 elif leaf.type in OPENING_BRACKETS:
1048 # There are brackets we can further split on.
1049 seen_other_brackets = True
1054 def line_to_string(line: Line) -> str:
1055 """Returns the string representation of @line.
1057 WARNING: This is known to be computationally expensive.
1059 return str(line).strip("\n")