style: fix lints (#264)

Cargo.toml

@@ -9,6 +9,13 @@ homepage = "https://github.com/ForwardSimulation/forrustts"
 repository = "https://github.com/ForwardSimulation/forrustts"
 keywords = ["simulation", "tree_sequences", "tskit", "population_genetics"]
 
+# Added for rust >= 1.80.0
+[lints.rust]
+unexpected_cfgs = { level = "warn", check-cfg = ['cfg(doc_cfg)'] }
+
+[package.metadata.docs.rs]
+all-features = true
+rustdoc-args = ["--cfg", "doc_cfg"]
 
 [workspace]
 members = [

forrustts-core/src/lib.rs

@@ -2,7 +2,6 @@
 
 #![warn(missing_docs)]
 #![warn(rustdoc::broken_intra_doc_links)]
-#![cfg_attr(doc_cfg, feature(doc_cfg))]
 
 use thiserror::Error;
 

forrustts-genetics/src/genetic_maps.rs

@@ -15,20 +15,24 @@ pub enum Breakpoint {
     IndependentAssortment(Position),
 }
 
+impl Breakpoint {
+    fn as_position(&self) -> Position {
+        match self {
+            Breakpoint::IndependentAssortment(p) => *p,
+            Breakpoint::Crossover(p) => *p,
+        }
+    }
+}
+
 impl From<Breakpoint> for Position {
     fn from(value: Breakpoint) -> Self {
-        match value {
-            Breakpoint::IndependentAssortment(p) => p,
-            Breakpoint::Crossover(p) => p,
-        }
+        value.as_position()
     }
 }
 
 impl PartialOrd for Breakpoint {
     fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        let left = Position::from(*self);
-        let right = Position::from(*other);
-        left.partial_cmp(&right)
+        Some(self.as_position().cmp(&other.as_position()))
     }
 }
 

forrustts-genetics/src/lib.rs

@@ -11,20 +11,20 @@ pub use genetic_maps::GeneticMapStatus;
 pub use genetic_maps::IndependentAssortment;
 pub use genetic_maps::PoissonCrossover;
 
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use rand::Rng;
-
-    struct MyGeneticMap {}
-
-    impl GenerateBreakpoints for MyGeneticMap {
-        fn generate_breakpoints<T: Rng>(&mut self, _rng: &mut T) {}
-        fn breakpoints(&self) -> &[Breakpoint] {
-            &[]
-        }
-    }
-}
+// #[cfg(test)]
+// mod tests {
+//     use super::*;
+//     use rand::Rng;
+//
+//     struct MyGeneticMap {}
+//
+//     impl GenerateBreakpoints for MyGeneticMap {
+//         fn generate_breakpoints<T: Rng>(&mut self, _rng: &mut T) {}
+//         fn breakpoints(&self) -> &[Breakpoint] {
+//             &[]
+//         }
+//     }
+// }
 
 #[test]
 fn test_poisson_crossover() {