clippy fixes

master
Jason Staten 6 years ago
parent f70bebd362
commit 8e3b00415d

@ -55,7 +55,7 @@ pub fn run(machine: Machine, code: Tape) -> Tape {
}; };
} }
return tape; tape
} }
#[test] #[test]

@ -2,12 +2,7 @@
#[allow(unused_imports)] #[allow(unused_imports)]
pub fn brute_force_search<T: PartialEq>(arr: &[T], el: T) -> bool { pub fn brute_force_search<T: PartialEq>(arr: &[T], el: T) -> bool {
for i in 0..arr.len() { arr.contains(&el)
if arr[i] == el {
return true;
}
}
return false;
} }
pub fn binary_search<T: Ord>(arr: &[T], el: T) -> bool { pub fn binary_search<T: Ord>(arr: &[T], el: T) -> bool {

@ -20,6 +20,10 @@ impl<T: PartialOrd> Heap<T> {
self.elements.len() self.elements.len()
} }
pub fn is_empty(&self) -> bool {
self.elements.is_empty()
}
pub fn push(&mut self, item: T) { pub fn push(&mut self, item: T) {
self.elements.push(item); self.elements.push(item);
let mut i = self.len() - 1; let mut i = self.len() - 1;
@ -30,7 +34,7 @@ impl<T: PartialOrd> Heap<T> {
} }
pub fn pop(&mut self) -> Option<T> { pub fn pop(&mut self) -> Option<T> {
if self.len() > 0 { if !self.is_empty() {
let result = self.elements.swap_remove(0); let result = self.elements.swap_remove(0);
self.heapify(0); self.heapify(0);
Some(result) Some(result)

@ -26,6 +26,10 @@ impl<T> Queue<T> {
self.length self.length
} }
pub fn is_empty(&self) -> bool {
self.length == 0
}
pub fn enqueue(&mut self, value: T) { pub fn enqueue(&mut self, value: T) {
let mut item = Box::new(Node { let mut item = Box::new(Node {
value, value,
@ -44,7 +48,7 @@ impl<T> Queue<T> {
self.rear = raw_rear; self.rear = raw_rear;
self.length = self.length + 1; self.length += 1;
} }
pub fn dequeue(&mut self) -> Option<T> { pub fn dequeue(&mut self) -> Option<T> {

Loading…
Cancel
Save