Finder found candidates cleanup

src/pip/_internal/index.py

@@ -276,9 +276,20 @@ def __init__(
     ):
         # type: (...) -> None
         self._candidates = candidates
-        self._specifier = specifier
-        self._prereleases = prereleases
         self._sort_key = sort_key
+        self._versions = {
+            str(v) for v in specifier.filter(
+                # We turn the version object into a str here because otherwise
+                # when we're debundled but setuptools isn't, Python will see
+                # packaging.version.Version and
+                # pkg_resources._vendor.packaging.version.Version as different
+                # types. This way we'll use a str as a common data interchange
+                # format. If we stop using the pkg_resources provided specifier
+                # and start using our own, we can drop the cast to str().
+                (str(c.version) for c in candidates),
+                prereleases=prereleases,
+            )
+        }
 
     def iter_all(self):
         # type: () -> Iterable[InstallationCandidate]
@@ -288,22 +299,10 @@ def iter_all(self):
 
     def iter_applicable(self):
         # type: () -> Iterable[InstallationCandidate]
-        """Iterate through candidates matching the given specifier.
+        """Iterate through candidates matching the desired versions.
         """
-        # Filter out anything which doesn't match our specifier.
-        versions = set(self._specifier.filter(
-            # We turn the version object into a str here because otherwise
-            # when we're debundled but setuptools isn't, Python will see
-            # packaging.version.Version and
-            # pkg_resources._vendor.packaging.version.Version as different
-            # types. This way we'll use a str as a common data interchange
-            # format. If we stop using the pkg_resources provided specifier
-            # and start using our own, we can drop the cast to str().
-            [str(c.version) for c in self._candidates],
-            prereleases=self._prereleases,
-        ))
-        # Again, converting to str to deal with debundling.
-        return (c for c in self._candidates if str(c.version) in versions)
+        # Again, converting version to str to deal with debundling.
+        return (c for c in self.iter_all() if str(c.version) in self._versions)
 
     def get_best(self):
         # type: () -> Optional[InstallationCandidate]
@@ -692,8 +691,8 @@ def find_all_candidates(self, project_name):
 
     def find_candidates(
         self,
-        project_name,   # type: str
-        specifier=specifiers.SpecifierSet(),  # type: specifiers.BaseSpecifier
+        project_name,       # type: str
+        specifier=None,     # type: Optional[specifiers.BaseSpecifier]
     ):
         """Find matches for the given project and specifier.
 
@@ -702,6 +701,8 @@ def find_candidates(
 
         Returns a `FoundCandidates` instance.
         """
+        if specifier is None:
+            specifier = specifiers.SpecifierSet()
         return FoundCandidates(
             self.find_all_candidates(project_name),
             specifier=specifier,
@@ -727,7 +728,8 @@ def find_requirement(self, req, upgrade):
         def _format_versions(cand_iter):
             # This repeated parse_version and str() conversion is needed to
             # handle different vendoring sources from pip and pkg_resources.
-            # If we stop using the pkg_resources provided specifier.
+            # If we stop using the pkg_resources provided specifier and start
+            # using our own, we can drop the cast to str().
             return ", ".join(sorted(
                 {str(c.version) for c in cand_iter},
                 key=parse_version,