Mercurial > public > mercurial-scm > hg
comparison mercurial/hg.py @ 8174:29bc5d18714a
hg: allow hg.parseurl(url, None)
In many places hg.parseurl is called with a url and "opts.get('rev')",
suggesting the second, optional argument can be None. Because opts['rev']
usually defaults to [] this never happens in practice.
However, extensions don't necessarily behave the same, but do copy this
pattern.
Also, include wider hg.parseurl tests, beyond a demonstration of the problem.
| author | Martijn Pieters <mj@zopatista.com> |
|---|---|
| date | Fri, 24 Apr 2009 18:17:42 +0200 |
| parents | b616f328af9f |
| children | 0f3b8404051b |
comparison
equal
deleted
inserted
replaced
| 8173:d3fb413667e5 | 8174:29bc5d18714a |
|---|---|
| 23 if '#' not in url: | 23 if '#' not in url: |
| 24 return url, (revs or None), revs and revs[-1] or None | 24 return url, (revs or None), revs and revs[-1] or None |
| 25 | 25 |
| 26 url, branch = url.split('#', 1) | 26 url, branch = url.split('#', 1) |
| 27 checkout = revs and revs[-1] or branch | 27 checkout = revs and revs[-1] or branch |
| 28 return url, revs + [branch], checkout | 28 return url, (revs or []) + [branch], checkout |
| 29 | 29 |
| 30 schemes = { | 30 schemes = { |
| 31 'bundle': bundlerepo, | 31 'bundle': bundlerepo, |
| 32 'file': _local, | 32 'file': _local, |
| 33 'http': httprepo, | 33 'http': httprepo, |