Page MenuHomePhabricator

manifest: move matches method to be outside the interface
ClosedPublic

Authored by durin42 on Wed, Feb 5, 5:34 PM.

Details

Summary

In order to adequately smoke out any legacy consumers of the method, we rename
it to _matches so it's clear that it's class-private. To my amazement, all
consumers of this method really only wanted matching filenames, not a full
filtered manifest.

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

durin42 created this revision.Wed, Feb 5, 5:34 PM
martinvonz requested changes to this revision.Fri, Feb 7, 5:08 PM
martinvonz added a subscriber: martinvonz.
martinvonz added inline comments.
mercurial/manifest.py
1207

This now checks match.always() recursively for every tree manifest. Seems wasteful, but maybe not noticeable? Could you test (or just revert that part of the patch)?

This revision now requires changes to proceed.Fri, Feb 7, 5:08 PM
durin42 updated this revision to Diff 20101.Mon, Feb 10, 7:49 PM
pulkit accepted this revision.Mon, Feb 10, 9:33 PM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.