changeset 42125:bc15e37ecc16

repoview: fix conditional around unserved changesets The conditional could lead to wrong computation since we have more unserved changesets than just the "secret" phase. In addition, now that we have efficient caching of phased changesets, we don't need the conditional anymore.
author Pierre-Yves David <pierre-yves.david@octobus.net>
date Sat, 13 Apr 2019 20:57:50 +0200
parents d6437f414437
children ef0e3cc684b3
files mercurial/repoview.py
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/repoview.py	Fri Apr 05 16:05:20 2019 +0300
+++ b/mercurial/repoview.py	Sat Apr 13 20:57:50 2019 +0200
@@ -93,8 +93,8 @@
     assert not repo.changelog.filteredrevs
     # fast path in simple case to avoid impact of non optimised code
     hiddens = filterrevs(repo, 'visible')
-    if phases.hassecret(repo):
-        secrets = repo._phasecache.getrevset(repo, phases.remotehiddenphases)
+    secrets = repo._phasecache.getrevset(repo, phases.remotehiddenphases)
+    if secrets:
         return frozenset(hiddens | frozenset(secrets))
     else:
         return hiddens