Skip to content

Commit

Permalink
Rename merge_dependency into merge_dependants
Browse files Browse the repository at this point in the history
  • Loading branch information
mpizenberg committed Dec 15, 2023
1 parent c646d3c commit 4c5800e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/internal/core.rs
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ impl<P: Package, VS: VersionSet, Priority: Ord + Clone> State<P, VS, Priority> {
.or_default();
if let Some((past, mergeed)) = deps_lookup.as_mut_slice().iter_mut().find_map(|past| {
self.incompatibility_store[id]
.merge_dependency(&self.incompatibility_store[*past])
.merge_dependants(&self.incompatibility_store[*past])
.map(|m| (past, m))
}) {
let new = self.incompatibility_store.alloc(mergeed);
Expand Down
2 changes: 1 addition & 1 deletion src/internal/incompatibility.rs
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ impl<P: Package, VS: VersionSet> Incompatibility<P, VS> {
///
/// It is a special case of prior cause computation where the unified package
/// is the common dependant in the two incompatibilities expressing dependencies.
pub fn merge_dependency(&self, other: &Self) -> Option<Self> {
pub fn merge_dependants(&self, other: &Self) -> Option<Self> {
// It is almost certainly a bug to call this method without checking that self is a dependency
debug_assert!(self.as_dependency().is_some());
// Check that both incompatibilities are of the shape p1 depends on p2,
Expand Down

0 comments on commit 4c5800e

Please sign in to comment.