psf / black / 28672992099

96%

master: 96%

Build Type

push

github

Committed by web-flow

Commit Message

batch --line-ranges node conversion to avoid quadratic removal (#5213)

* batch --line-ranges node conversion to avoid quadratic removal

* Fix decorator handling in batched --line-ranges conversion

Deferring the STANDALONE_COMMENT conversions broke single-line decorated
blocks. The per-node path relied on the decorator being collapsed in place
before the decorated node was converted: it re-inserted a NEWLINE between
the two and read the migrated prefix off the replaced first leaf. With the
conversions batched, the decorator is still its own child carrying its
NEWLINE, so re-inserting one doubled the blank line, and the enclosing
decorated conversion picked up an empty prefix after the decorator run had
already cleared it.

Drop the now-redundant NEWLINE insertion, remember each first leaf's taken
prefix so an inner and an outer conversion over the same leaf reuse it, and
zero that leaf's lineno to mirror the fresh line-0 STANDALONE_COMMENT the
in-place path used to leave behind. Fixes line_ranges_decorator_edge_case.

---------

Co-authored-by: cobalt <61329810+cobaltt7@users.noreply.github.com>

Coverage Stats

5841 of 6165 branches covered (94.74%)

70 of 71 new or added lines in 1 file covered. (98.59%)

1 existing line in 1 file now uncovered.

8677 of 9052 relevant lines covered (95.86%)

5.74 hits per line