From 0ede27e9272de4b3e21a08f1ee2d2f90cbd82ede Mon Sep 17 00:00:00 2001 From: electria Date: Tue, 20 Oct 2026 19:26:14 -0700 Subject: [PATCH] refactor: rename confusing index variable --- src/nodes/eval.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/nodes/eval.rs b/src/nodes/eval.rs index 25fe190..33c606b 100644 --- a/src/nodes/eval.rs +++ b/src/nodes/eval.rs @@ -91,23 +91,23 @@ fn collapse_toplevel(nodes: &mut Vec, mut end: usize) -> Result<(), Error> macro_rules! collapse_operator { ($operator:expr) => { - while let Some(operator) = nodes[start..end] + while let Some(index) = nodes[start..end] .iter() .position(|n| *n == Node::Operator($operator)) { - let lhs = match nodes.remove(start + operator - 1) { + let lhs = match nodes.remove(start + index - 1) { Node::Number(n) => n, n => return Err(Error::InvalidOperand(n)), }; end -= 1; - match nodes.get_mut(start + operator) { - None => return Err(Error::OutOfBounds(Nodes(nodes.clone()), start + operator)), + match nodes.get_mut(start + index) { + None => return Err(Error::OutOfBounds(Nodes(nodes.clone()), start + index)), Some(Node::Number(n)) => $operator.apply_to(n, lhs), Some(n) => return Err(Error::InvalidOperand(n.clone())), }; - nodes.remove(start + operator - 1); + nodes.remove(start + index - 1); end -= 1; } };