Mercurial > hg
changeset 31707:9cd640e5c1ba
tags: reorder argument of '_updatetags'
We move all arguments related to tagtype to the end, together. This will allow
us to make these arguments optional and reuse of this logic for callers that do
not care about the tag types.
author | Pierre-Yves David <pierre-yves.david@ens-lyon.org> |
---|---|
date | Tue, 28 Mar 2017 07:38:10 +0200 |
parents | 63d4deda1b31 |
children | d0e7c70f14b7 |
files | mercurial/tags.py |
diffstat | 1 files changed, 4 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/tags.py Tue Mar 28 06:13:49 2017 +0200 +++ b/mercurial/tags.py Tue Mar 28 07:38:10 2017 +0200 @@ -97,7 +97,7 @@ # XXX is this really 100% correct? are there oddball special # cases where a global tag should outrank a local tag but won't, # because cachetags does not contain rank info? - _updatetags(cachetags, 'global', alltags, tagtypes) + _updatetags(cachetags, alltags, 'global', tagtypes) return alltags, tagtypes seen = set() # set of fnode @@ -115,7 +115,7 @@ fctx = fctx.filectx(fnode) filetags = _readtags(ui, repo, fctx.data().splitlines(), fctx) - _updatetags(filetags, 'global', alltags, tagtypes) + _updatetags(filetags, alltags, 'global', tagtypes) # and update the cache (if necessary) if shouldwrite: @@ -145,7 +145,7 @@ except (LookupError, ValueError): del filetags[t] - _updatetags(filetags, "local", alltags, tagtypes) + _updatetags(filetags, alltags, 'local', tagtypes) def _readtaghist(ui, repo, lines, fn, recode=None, calcnodelines=False): '''Read tag definitions from a file (or any source of lines). @@ -223,7 +223,7 @@ newtags[tag] = (taghist[-1], taghist[:-1]) return newtags -def _updatetags(filetags, tagtype, alltags, tagtypes): +def _updatetags(filetags, alltags, tagtype, tagtypes): '''Incorporate the tag info read from one file into the two dictionaries, alltags and tagtypes, that contain all tag info (global across all heads plus local).'''