feat: make load from file generic over AsRef<str>

* for TableCollection
* for TreeSequence

Author: molpopgen

src/table_collection.rs

@@ -138,14 +138,14 @@ impl TableCollection {
     ///
     /// This function allocates a `CString` to pass the file name to the C API.
     /// A panic will occur if the system runs out of memory.
-    pub fn new_from_file(filename: &str) -> Result<Self, TskitError> {
+    pub fn new_from_file(filename: impl AsRef<str>) -> Result<Self, TskitError> {
         // Arbitrary sequence_length.
         let mut tables = match TableCollection::new(1.0) {
             Ok(t) => (t),
             Err(e) => return Err(e),
         };
 
-        let c_str = std::ffi::CString::new(filename).unwrap();
+        let c_str = std::ffi::CString::new(filename.as_ref()).unwrap();
         let rv = unsafe {
             ll_bindings::tsk_table_collection_load(
                 tables.as_mut_ptr(),

src/trees.rs

@@ -1042,8 +1042,8 @@ impl TreeSequence {
     }
 
     /// Load from a file.
-    pub fn load(filename: &str) -> Result<Self, TskitError> {
-        let tables = TableCollection::new_from_file(filename)?;
+    pub fn load(filename: impl AsRef<str>) -> Result<Self, TskitError> {
+        let tables = TableCollection::new_from_file(filename.as_ref())?;
 
         Self::new(tables, TreeSequenceFlags::default())
     }