fix clippy warnings that showed up in rust 1.54 (#153)

Author: molpopgen

examples/forward_simulation.rs

@@ -372,8 +372,8 @@ fn runsim(params: &SimParams) -> tskit::TableCollection {
 
     for step in (0..params.nsteps).rev() {
         parents.clear();
-        death_and_parents(&alive, &params, &mut parents, &mut rng);
-        births(&parents, &params, step, &mut tables, &mut alive, &mut rng);
+        death_and_parents(&alive, params, &mut parents, &mut rng);
+        births(&parents, params, step, &mut tables, &mut alive, &mut rng);
         let remainder = step % params.simplification_interval;
         match step < params.nsteps && remainder == 0 {
             true => {

examples/tree_traversals.rs

@@ -42,8 +42,8 @@ fn main() {
     let mut tree_iterator = treeseq.tree_iterator(tskit::TreeFlags::default()).unwrap();
 
     while let Some(tree) = tree_iterator.next() {
-        traverse_upwards(&tree);
-        traverse_upwards_with_iterator(&tree);
-        preorder_traversal(&tree);
+        traverse_upwards(tree);
+        traverse_upwards_with_iterator(tree);
+        preorder_traversal(tree);
     }
 }

src/edge_table.rs

@@ -134,7 +134,7 @@ impl<'a> EdgeTable<'a> {
     /// The value of the iterator is [`EdgeTableRow`].
     ///
     pub fn iter(&self) -> EdgeTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&EdgeTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&EdgeTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/individual_table.rs

@@ -27,7 +27,7 @@ impl PartialEq for IndividualTableRow {
                             false
                         } else {
                             for (i, j) in a.iter().enumerate() {
-                                if !crate::util::f64_partial_cmp_equal(&j, &b[i]) {
+                                if !crate::util::f64_partial_cmp_equal(j, &b[i]) {
                                     return false;
                                 }
                             }
@@ -164,7 +164,7 @@ impl<'a> IndividualTable<'a> {
     /// The value of the iterator is [`IndividualTableRow`].
     ///
     pub fn iter(&self) -> IndividualTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&IndividualTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&IndividualTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/migration_table.rs

@@ -173,7 +173,7 @@ impl<'a> MigrationTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`MigrationTableRow`].
     pub fn iter(&self) -> MigrationTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&MigrationTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&MigrationTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/mutation_table.rs

@@ -49,7 +49,7 @@ impl<'a> Iterator for MutationTableRefIterator<'a> {
     type Item = MutationTableRow;
 
     fn next(&mut self) -> Option<Self::Item> {
-        let rv = make_mutation_table_row(&self.table, self.pos);
+        let rv = make_mutation_table_row(self.table, self.pos);
         self.pos += 1;
         rv
     }
@@ -164,7 +164,7 @@ impl<'a> MutationTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`MutationTableRow`].
     pub fn iter(&self) -> MutationTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&MutationTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&MutationTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/node_table.rs

@@ -170,7 +170,7 @@ impl<'a> NodeTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`NodeTableRow`].
     pub fn iter(&self) -> NodeTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&NodeTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&NodeTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/population_table.rs

@@ -37,7 +37,7 @@ impl<'a> Iterator for PopulationTableRefIterator<'a> {
     type Item = PopulationTableRow;
 
     fn next(&mut self) -> Option<Self::Item> {
-        let rv = make_population_table_row(&self.table, self.pos);
+        let rv = make_population_table_row(self.table, self.pos);
         self.pos += 1;
         rv
     }
@@ -83,7 +83,7 @@ impl<'a> PopulationTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`PopulationTableRow`].
     pub fn iter(&self) -> PopulationTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&PopulationTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&PopulationTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/provenance.rs

@@ -269,7 +269,7 @@ impl<'a> ProvenanceTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`ProvenanceTableRow`].
     pub fn iter(&self) -> ProvenanceTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&ProvenanceTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&ProvenanceTable<'a>>(self)
     }
 }
 
@@ -301,7 +301,7 @@ mod test_provenance_tables {
         let records = vec!["banana".to_string(), "split".to_string()];
         let mut tables = make_empty_table_collection(1.);
         for (i, r) in records.iter().enumerate() {
-            let row_id = tables.add_provenance(&r).unwrap();
+            let row_id = tables.add_provenance(r).unwrap();
             assert!(row_id == ProvenanceId(i as crate::tsk_id_t));
             assert_eq!(tables.provenances().record(row_id).unwrap(), *r);
         }

src/site_table.rs

@@ -42,7 +42,7 @@ impl<'a> Iterator for SiteTableRefIterator<'a> {
     type Item = SiteTableRow;
 
     fn next(&mut self) -> Option<Self::Item> {
-        let rv = make_site_table_row(&self.table, self.pos);
+        let rv = make_site_table_row(self.table, self.pos);
         self.pos += 1;
         rv
     }
@@ -121,7 +121,7 @@ impl<'a> SiteTable<'a> {
     /// Return an iterator over rows of the table.
     /// The value of the iterator is [`SiteTableRow`].
     pub fn iter(&self) -> SiteTableRefIterator {
-        crate::table_iterator::make_table_iterator::<&SiteTable<'a>>(&self)
+        crate::table_iterator::make_table_iterator::<&SiteTable<'a>>(self)
     }
 
     /// Return row `r` of the table.

src/table_collection.rs

@@ -1192,10 +1192,10 @@ mod test {
             .unwrap();
         tables.add_edge(0., tables.sequence_length(), 1, 0).unwrap();
         tables
-            .dump(&treefile, TableOutputOptions::default())
+            .dump(treefile, TableOutputOptions::default())
             .unwrap();
 
-        let tables2 = TableCollection::new_from_file(&treefile).unwrap();
+        let tables2 = TableCollection::new_from_file(treefile).unwrap();
         assert!(tables.equals(&tables2, TableEqualityOptions::default()));
 
         std::fs::remove_file(&treefile).unwrap();
@@ -1266,7 +1266,7 @@ mod test {
             Some(x) => {
                 assert_eq!(x.len(), location.len());
                 for (i, l) in x.iter().enumerate() {
-                    assert!(crate::util::f64_partial_cmp_equal(&l, &location[i]));
+                    assert!(crate::util::f64_partial_cmp_equal(l, &location[i]));
                 }
             }
             None => panic!("expected some locations"),

src/test_simplification.rs

@@ -34,7 +34,7 @@ mod tests {
         let ts = treeseq_from_small_table_collection_two_trees();
         let samples = ts.sample_nodes();
         let (_, idmap_option) = ts
-            .simplify(&samples, SimplificationOptions::default(), true)
+            .simplify(samples, SimplificationOptions::default(), true)
             .unwrap();
         assert!(idmap_option.is_some());
         let idmap = idmap_option.unwrap();

src/trees.rs

@@ -440,7 +440,7 @@ impl Tree {
     ///
     /// [`TskitError::IndexError`] if `u` is out of range.
     pub fn children(&self, u: NodeId) -> Result<impl Iterator<Item = NodeId> + '_, TskitError> {
-        ChildIterator::new(&self, u)
+        ChildIterator::new(self, u)
     }
     /// Return an [`Iterator`] over the sample nodes descending from node `u`.
     ///
@@ -491,7 +491,7 @@ impl Tree {
         order: NodeTraversalOrder,
     ) -> Box<dyn Iterator<Item = NodeId> + '_> {
         match order {
-            NodeTraversalOrder::Preorder => Box::new(PreorderNodeIterator::new(&self)),
+            NodeTraversalOrder::Preorder => Box::new(PreorderNodeIterator::new(self)),
         }
     }
 
@@ -585,7 +585,7 @@ impl streaming_iterator::StreamingIterator for Tree {
 
     fn get(&self) -> Option<&Tree> {
         match self.advanced {
-            true => Some(&self),
+            true => Some(self),
             false => None,
         }
     }