finish the refactor.

Author: molpopgen

src/trees.rs

@@ -61,7 +61,7 @@ impl Tree {
         let flags = flags.into();
 
         // SAFETY: this is the type we want :)
-        let mut temp = unsafe {
+        let temp = unsafe {
             libc::malloc(std::mem::size_of::<ll_bindings::tsk_tree_t>())
                 as *mut ll_bindings::tsk_tree_t
         };
@@ -79,6 +79,7 @@ impl Tree {
         }
         // Gotta ask Jerome about this one--why isn't this handled in tsk_tree_init??
         if !flags.contains(TreeFlags::NO_SAMPLE_COUNTS) {
+            // SAFETY: nobody is null here.
             rv = unsafe {
                 ll_bindings::tsk_tree_set_tracked_samples(
                     tree.as_mut(),
@@ -89,7 +90,7 @@ impl Tree {
         }
 
         let num_nodes = unsafe { (*(*ts.as_ptr()).tables).nodes.num_rows };
-        let api = TreeInterface::new(tree.as_ref(), num_nodes, num_nodes + 1, flags);
+        let api = TreeInterface::new(nonnull, num_nodes, num_nodes + 1, flags);
         handle_tsk_return_value!(
             rv,
             Tree {