Patchwork D1885: commands: rewrite legacy ternary operator hack using modern syntax

login
register
mail settings
Submitter phabricator
Date Jan. 19, 2018, 8:12 p.m.
Message ID <28f68cc3d2737cf8437664263c1c3323@localhost.localdomain>
Download mbox | patch
Permalink /patch/26971/
State Not Applicable
Headers show

Comments

phabricator - Jan. 19, 2018, 8:12 p.m.
durin42 updated this revision to Diff 4946.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D1885?vs=4875&id=4946

REVISION DETAIL
  https://phab.mercurial-scm.org/D1885

AFFECTED FILES
  mercurial/commands.py

CHANGE DETAILS




To: durin42, #hg-reviewers, pulkit
Cc: mercurial-devel

Patch

diff --git a/mercurial/commands.py b/mercurial/commands.py
--- a/mercurial/commands.py
+++ b/mercurial/commands.py
@@ -1219,7 +1219,7 @@ 
             raise error.Abort(_("--base is incompatible with specifying "
                                "a destination"))
         common = [repo.lookup(rev) for rev in base]
-        heads = revs and map(repo.lookup, revs) or None
+        heads = map(repo.lookup, revs) if revs else None
         outgoing = discovery.outgoing(repo, common, heads)
     else:
         dest = ui.expandpath(dest or 'default-push', dest or 'default')