Page MenuHomePhabricator

copies: document cases in _chain()
ClosedPublic

Authored by martinvonz on Apr 18 2019, 12:55 PM.

Details

Summary

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Apr 18 2019, 12:55 PM
martinvonz added inline comments.Apr 18 2019, 1:49 PM
mercurial/copies.py
120–125

I just realized that using a, b, c as filenames is confusing since a and b are also the names of the copy dicts. I'll rename the files to x, y, z.

martinvonz updated this revision to Diff 14846.Apr 18 2019, 1:58 PM

I think the rest of the stack from this patch are independent of the ones before, so if reviewers are intimidated by D6255, they should still be able to take this section of the stack.

martinvonz updated this revision to Diff 14948.Apr 28 2019, 3:00 AM
martinvonz updated this revision to Diff 14969.Apr 29 2019, 6:52 PM
This revision was automatically updated to reflect the committed changes.