( )⚙ D6344 copies: remove redundant filtering of ping-pong renames in _chain()

This is an archive of the discontinued Mercurial Phabricator instance.

copies: remove redundant filtering of ping-pong renames in _chain()
ClosedPublic

Authored by martinvonz on May 5 2019, 1:30 AM.

Details

Summary

We already handle the ping-pong rename case in the filtering step, so
there's very little point in doing it in the chaining loop (ping-pong
renames are very rare, so I'm not worried about the cost of adding it
and then removing it again).

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.May 5 2019, 1:30 AM
This revision was automatically updated to reflect the committed changes.