diff mercurial/revset.py @ 26093:3e9049876ace

revset: gratuitous code move in '_children' As 'cs' is empty as the time of the conditional, we can just return an empty 'baseset' and create the variable later.
author Pierre-Yves David <pierre-yves.david@fb.com>
date Thu, 11 Jun 2015 14:27:52 -0700 (2015-06-11)
parents f93ff3ab8d14
children c1d163ce7394
line wrap: on
line diff
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -614,9 +614,9 @@
     return subset.filter(matches)
 
 def _children(repo, narrow, parentset):
+    if not parentset:
+        return baseset()
     cs = set()
-    if not parentset:
-        return baseset(cs)
     pr = repo.changelog.parentrevs
     # XXX this should be 'parentset.min()' assuming 'parentset' is a smartset
     # (and if it is not, it should.)