okay

Author: molpopgen

src/sys/tree.rs

@@ -77,9 +77,8 @@ impl<'treeseq> LLTree<'treeseq> {
         self.inner.as_ref()
     }
 
-    pub fn left_sib<N: Into<tsk_id_t> + Copy>(&self, u: N) -> Option<tsk_id_t> {
-        todo!("doc SAFETY");
-        super::tsk_column_access::<tsk_id_t, _, _, _>(
+    pub fn left_sib(&self, u: NodeId) -> Option<NodeId> {
+        super::tsk_column_access::<NodeId, _, _, _>(
             u.into(),
             self.as_ref().left_sib,
             unsafe {

src/trees/tree.rs

@@ -101,6 +101,13 @@ impl<'treeseq> Tree<'treeseq> {
     pub fn virtual_root(&self) -> NodeId {
         self.inner.virtual_root().into()
     }
+
+    /// Get the left sib of node `u`.
+    ///
+    /// Returns `None` if `u` is out of range.
+    pub fn left_sib<N: Into<NodeId> + Copy>(&self, u: N) -> Option<NodeId> {
+        self.inner.left_sib(u.into()).map(|x| x.into())
+    }
 }
 
 impl<'ts> streaming_iterator::StreamingIterator for Tree<'ts> {