Remove unused abs_x/_y variables from round_layout (#319)

These variables are unused, but the compiler doesn't warn about them
because they're passed into a recursive call.

320118ed7bae2764c9fc634ed0827927113abd05 · by Benjamin Woodruff · 2023-01-05T00:56:34Z (3 years ago) · parent 70d495a7b · browse the tree at this commit

Semantic delta · 2

src/compute/mod.rs

body changed fn compute_layout pub
signature changed fn round_layout

1 files · +3 −5

src/compute/mod.rs +3 −5

diff --git a/src/compute/mod.rs b/src/compute/mod.rs
index d130ccdf0..4c0357a58 100644
--- a/src/compute/mod.rs
+++ b/src/compute/mod.rs
@@ -40,7 +40,7 @@ pub fn compute_layout(
     *tree.layout_mut(root) = layout;
 
     // Recursively round the layout's of this node and all children
-    round_layout(tree, root, 0.0, 0.0);
+    round_layout(tree, root);
 
     Ok(())
 }
@@ -231,10 +231,8 @@ fn perform_hidden_layout(tree: &mut impl LayoutTree, node: Node) -> Size<f32> {
 }
 
 /// Rounds the calculated [`NodeData`] according to the spec
-fn round_layout(tree: &mut impl LayoutTree, root: Node, abs_x: f32, abs_y: f32) {
+fn round_layout(tree: &mut impl LayoutTree, root: Node) {
     let layout = tree.layout_mut(root);
-    let abs_x = abs_x + layout.location.x;
-    let abs_y = abs_y + layout.location.y;
 
     layout.location.x = round(layout.location.x);
     layout.location.y = round(layout.location.y);
@@ -245,7 +243,7 @@ fn round_layout(tree: &mut impl LayoutTree, root: Node, abs_x: f32, abs_y: f32)
     // Satisfy the borrow checker here by re-indexing to shorten the lifetime to the loop scope
     for x in 0..tree.child_count(root) {
         let child = tree.child(root, x);
-        round_layout(tree, child, abs_x, abs_y);
+        round_layout(tree, child);
     }
 }