Minor cleanup

TODO.md

@@ -40,7 +40,6 @@
 ## Internals
  - Non-recursive treeify
  - Faster factorial function. Maybe use gamma instead?
- - Remove extra calls to `.clone()` in quantity module
  - Arbitrary precision float (rug doesn't offer arbitrary exponents)
  - Backend-independent colorful printing
 

src/evaluate/operator.rs

@@ -38,7 +38,7 @@ pub fn eval_operator(g: &Expression, context: &mut Context) -> Result<Option<Exp
 			let mut loc: LineLocation;
 			if let Expression::Quantity(l, s) = &args[0] {
 				sum = s.clone();
-				loc = l.clone();
+				loc = *l;
 			} else { return Ok(None); };
 
 
@@ -111,7 +111,7 @@ pub fn eval_operator(g: &Expression, context: &mut Context) -> Result<Option<Exp
 			let mut loc: LineLocation;
 			if let Expression::Quantity(l, s) = &args[0] {
 				prod = s.clone();
-				loc = l.clone();
+				loc = *l;
 			} else { return Ok(None); };
 
 			let mut i: usize = 1;

src/parser/expression/expression.rs

@@ -127,7 +127,7 @@ impl Expression {
 			| Expression::Constant(l, _)
 			| Expression::Variable(l, _)
 			| Expression::Operator(l, _,_)
-			=> { l.clone() }
+			=> { *l }
 		}
 	}
 

src/parser/stage/find_subs.rs

@@ -84,7 +84,7 @@ pub fn find_subs(
 		} else {
 			let target = target.unwrap();
 			let l = t.get_mut_line_location();
-			r.push_back((l.clone(), String::from(target)));
+			r.push_back((*l, String::from(target)));
 
 			let old_len = l.len;
 			let new_len = target.chars().count();

src/parser/stage/groupify.rs

@@ -195,7 +195,7 @@ pub fn groupify(
 	(LineLocation, ParserError)
 > {
 
-	let last_linelocation = g.back().unwrap().get_line_location().clone();
+	let last_linelocation: LineLocation = *g.back().unwrap().get_line_location();
 
 	// Vector of grouping levels
 	let mut levels: Vec<(LineLocation, VecDeque<Token>)> = Vec::with_capacity(8);