From a5f1a4ea81b43d1e1647af0e1d1e02c22fad2155 Mon Sep 17 00:00:00 2001 From: Todd Gamblin Date: Sat, 15 Mar 2025 09:37:37 -0700 Subject: [PATCH] Spec: Remove unused methods on `_EdgeMap` `_EdgeMap` implements `_cmp_iter` for `lazy_lexicographic_ordering` but it's never used or tested. Remove it. Signed-off-by: Todd Gamblin --- lib/spack/spack/spec.py | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/lib/spack/spack/spec.py b/lib/spack/spack/spec.py index 01fb7f18d5d..2d966309414 100644 --- a/lib/spack/spack/spec.py +++ b/lib/spack/spack/spec.py @@ -961,7 +961,6 @@ def _sort_by_dep_types(dspec: DependencySpec): return dspec.depflag -@lang.lazy_lexicographic_ordering class _EdgeMap(collections.abc.Mapping): """Represent a collection of edges (DependencySpec objects) in the DAG. @@ -999,21 +998,6 @@ def add(self, edge: DependencySpec) -> None: def __str__(self) -> str: return f"{{deps: {', '.join(str(d) for d in sorted(self.values()))}}}" - def _cmp_iter(self): - for item in sorted(itertools.chain.from_iterable(self.edges.values())): - yield item - - def copy(self): - """Copies this object and returns a clone""" - clone = type(self)() - clone.store_by_child = self.store_by_child - - # Copy everything from this dict into it. - for dspec in itertools.chain.from_iterable(self.values()): - clone.add(dspec.copy()) - - return clone - def select( self, *,