Skip to content
This repository was archived by the owner on Dec 9, 2025. It is now read-only.

Commit c075611

Browse files
committed
rename get ref to take ref
1 parent e101300 commit c075611

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

optd/src/memo/memory/helpers.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ impl MemoryMemo {
3434
PhysicalExpressionId(id)
3535
}
3636

37-
/// Gets the set of [`LogicalExpressionId`] that reference a group, mapped to their
37+
/// Takes the set of [`LogicalExpressionId`] that reference a group, mapped to their
3838
/// representatives.
39-
fn get_referencing_expr_set(&mut self, group_id: GroupId) -> HashSet<LogicalExpressionId> {
39+
fn take_referencing_expr_set(&mut self, group_id: GroupId) -> HashSet<LogicalExpressionId> {
4040
self.group_referencing_exprs_index
4141
.remove(&group_id)
4242
.unwrap_or_default()
@@ -314,8 +314,8 @@ impl MemoryMemoHelpers for MemoryMemo {
314314
new_group_id: GroupId,
315315
) -> Result<Vec<(GroupId, GroupId)>, Infallible> {
316316
// Collect expressions referencing the merged groups.
317-
let expr_set_1 = self.get_referencing_expr_set(group_id_1);
318-
let expr_set_2 = self.get_referencing_expr_set(group_id_2);
317+
let expr_set_1 = self.take_referencing_expr_set(group_id_1);
318+
let expr_set_2 = self.take_referencing_expr_set(group_id_2);
319319

320320
// Combine into a single set.
321321
let mut referenced_exprs = expr_set_1;

0 commit comments

Comments
 (0)