From dbf3536ce44c037751e10840b9d02e8604d87f43 Mon Sep 17 00:00:00 2001
From: "Kevin R. Thornton" <krthornt@uci.edu>
Date: Thu, 16 Mar 2023 05:32:47 -0700
Subject: [PATCH] refactor!: TableCollection::simplify now takes &[NodeId]

Previous implementation took &[N] where N: Into<NodeId>,
which was unsound in this case.
---
 src/table_collection.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/table_collection.rs b/src/table_collection.rs
index bcbe21aed..3305a0224 100644
--- a/src/table_collection.rs
+++ b/src/table_collection.rs
@@ -763,9 +763,9 @@ impl TableCollection {
     ///   in length to the input node table.  For each input node,
     ///   this vector either contains the node's new index or [`NodeId::NULL`]
     ///   if the input node is not part of the simplified history.
-    pub fn simplify<N: Into<NodeId>, O: Into<SimplificationOptions>>(
+    pub fn simplify<O: Into<SimplificationOptions>>(
         &mut self,
-        samples: &[N],
+        samples: &[NodeId],
         options: O,
         idmap: bool,
     ) -> Result<Option<&[NodeId]>, TskitError> {