# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 2024-02-21 12:01:25
# Node ID 0c04074f5414c773cadb42ed9483b92bb0ab93d4
# Parent  1df8d84e7c990e58e6d32eb634ea2de3025e6422

phases: drop set building in `hasnonpublicphases`

We don't actually use the set, so why do we ensure they are built?

(we should also clean up the use of repository argument but that's a quest for later).

diff --git a/mercurial/phases.py b/mercurial/phases.py
--- a/mercurial/phases.py
+++ b/mercurial/phases.py
@@ -382,8 +382,7 @@ class phasecache:
 
     def hasnonpublicphases(self, repo: "localrepo.localrepository") -> bool:
         """detect if there are revisions with non-public phase"""
-        repo = repo.unfiltered()
-        self._ensure_phase_sets(repo)
+        # XXX deprecate the unused repo argument
         return any(
             revs for phase, revs in self._phaseroots.items() if phase != public
         )