# HG changeset patch # User Erik Zielke # Date 1285247890 -7200 # Node ID dece1f46f7a29461dc4a87a4b0bd066320a27766 # Parent c87216e5e43ebab0982c1cdc303948534bc050ec mq: removed return from callers to patchheader.qseries Some callers to patchheader.qseries returned the value further, but pathcheader.qseries does not explicitly return anything. It was confusing this was returned further diff -r c87216e5e43e -r dece1f46f7a2 hgext/mq.py --- a/hgext/mq.py Thu Sep 23 16:56:32 2010 +0200 +++ b/hgext/mq.py Thu Sep 23 15:18:10 2010 +0200 @@ -1778,8 +1778,9 @@ else: start = 0 - return q.qseries(repo, length=end, start=start, status='A', - summary=opts.get('summary')) + q.qseries(repo, length=end, start=start, status='A', + summary=opts.get('summary')) + def unapplied(ui, repo, patch=None, **opts): """print the patches not yet applied @@ -1799,8 +1800,8 @@ return 1 length = opts.get('first') and 1 or None - return q.qseries(repo, start=start, length=length, status='U', - summary=opts.get('summary')) + q.qseries(repo, start=start, length=length, status='U', + summary=opts.get('summary')) def qimport(ui, repo, *filename, **opts): """import a patch @@ -1976,8 +1977,8 @@ q = repo.mq t = q.applied and q.series_end(True) or 0 if t: - return q.qseries(repo, start=t - 1, length=1, status='A', - summary=opts.get('summary')) + q.qseries(repo, start=t - 1, length=1, status='A', + summary=opts.get('summary')) else: ui.write(_("no patches applied\n")) return 1 @@ -1991,7 +1992,7 @@ if end == len(q.series): ui.write(_("all patches applied\n")) return 1 - return q.qseries(repo, start=end, length=1, summary=opts.get('summary')) + q.qseries(repo, start=end, length=1, summary=opts.get('summary')) def prev(ui, repo, **opts): """print the name of the previous patch @@ -2005,8 +2006,8 @@ if not l: ui.write(_("no patches applied\n")) return 1 - return q.qseries(repo, start=l - 2, length=1, status='A', - summary=opts.get('summary')) + q.qseries(repo, start=l - 2, length=1, status='A', + summary=opts.get('summary')) def setupheaderopts(ui, opts): if not opts.get('user') and opts.get('currentuser'):