Add stomp() method to CoordinateSet

Author: busstoptaktik

src/coordinate/mod.rs

@@ -87,10 +87,20 @@ impl<T> CoordinateMetadata for T where T: ?Sized {}
 /// access any user provided data model by iterating over its elements,
 /// represented as a `Coor4D`
 pub trait CoordinateSet: CoordinateMetadata {
+    /// Number of coordinate tuples in the set
     fn len(&self) -> usize;
+    /// Access the `index`th coordinate tuple
     fn get_coord(&self, index: usize) -> Coor4D;
+    /// Overwrite the `index`th coordinate tuple
     fn set_coord(&mut self, index: usize, value: &Coor4D);
     fn is_empty(&self) -> bool {
         self.len() == 0
     }
+    /// Set all coordinate tuples in the set to NaN
+    fn stomp(&mut self) {
+        let nanny = Coor4D::nan();
+        for i in 0..self.len() {
+            self.set_coord(i, &nanny);
+        }
+    }
 }

src/inner_op/stack.rs

@@ -269,10 +269,7 @@ fn stack_flip(stack: &mut [Vec<f64>], operands: &mut dyn CoordinateSet, args: &[
     // In case of underflow, we stomp on all input coordinates
     if stack_depth < number_of_flips {
         warn!("Stack flip underflow in pipeline");
-        let nanny = Coor4D::nan();
-        for i in 0..number_of_operands {
-            operands.set_coord(i, &nanny);
-        }
+        operands.stomp();
         return 0;
     }
 
@@ -311,10 +308,7 @@ fn stack_roll(stack: &mut Vec<Vec<f64>>, operands: &mut dyn CoordinateSet, args:
 
     if m > depth {
         warn!("Roll too deep");
-        let nanny = Coor4D::nan();
-        for i in 0..operands.len() {
-            operands.set_coord(i, &nanny);
-        }
+        operands.stomp();
         return 0;
     }
 
@@ -335,10 +329,7 @@ fn stack_pop(stack: &mut Vec<Vec<f64>>, operands: &mut dyn CoordinateSet, args:
     // In case of underflow, we stomp on all input coordinates
     if stack_depth < number_of_pops {
         warn!("Stack underflow in pipeline");
-        let nanny = Coor4D::nan();
-        for i in 0..number_of_operands {
-            operands.set_coord(i, &nanny);
-        }
+        operands.stomp();
         return 0;
     }