# HG changeset patch # User Martin von Zweigbergk # Date 2014-11-13 06:20:36 # Node ID dd3f857598a0625305b106323544df425efeaf84 # Parent 3f269bd4826cccd22e238c8efca345cc7a345bf9 context.status: pass status tuple into _buildstatus By passing a status tuple (instead of the current list), we can access the status fields by name and make it a little more readable. diff --git a/mercurial/context.py b/mercurial/context.py --- a/mercurial/context.py +++ b/mercurial/context.py @@ -108,7 +108,7 @@ class basectx(object): mf2 = self._manifestmatches(match, s) modified, added, clean = [], [], [] - deleted, unknown, ignored = s[3], s[4], s[5] + deleted, unknown, ignored = s.deleted, s.unknown, s.ignored deletedset = set(deleted) withflags = mf1.withflags() | mf2.withflags() for fn, mf2node in mf2.iteritems(): @@ -301,7 +301,7 @@ class basectx(object): ctx1, ctx2 = ctx2, ctx1 match = ctx2._matchstatus(ctx1, match) - r = [[], [], [], [], [], [], []] + r = scmutil.status([], [], [], [], [], [], []) r = ctx2._buildstatus(ctx1, r, match, listignored, listclean, listunknown) @@ -1389,11 +1389,10 @@ class workingctx(committablectx): need to build a manifest and return what matches. """ mf = self._repo['.']._manifestmatches(match, s) - modified, added, removed = s[0:3] - for f in modified + added: + for f in s.modified + s.added: mf[f] = None mf.setflag(f, self.flags(f)) - for f in removed: + for f in s.removed: if f in mf: del mf[f] return mf