code review: remove unsafe on borrow checker constructor

crates/wiggle/test-helpers/src/borrow.rs

@@ -12,7 +12,7 @@ impl BorrowChecker {
     /// `GuestSlice` and `GuestStr` structs, which implement `std::ops::Deref` and
     /// `std::ops::DerefMut`. It also enforces that `GuestPtr::read` and `GuestPtr::write` do not
     /// access memory with an outstanding borrow.
-    pub unsafe fn new() -> Self {
+    pub fn new() -> Self {
         BorrowChecker {
             bc: RefCell::new(InnerBorrowChecker::new()),
         }

crates/wiggle/test-helpers/src/lib.rs

@@ -60,7 +60,7 @@ impl HostMemory {
             buffer: HostBuffer {
                 cell: UnsafeCell::new([0; 4096]),
             },
-            bc: unsafe { BorrowChecker::new() },
+            bc: BorrowChecker::new(),
         }
     }
 

crates/wiggle/wasmtime/src/borrow.rs

@@ -12,7 +12,7 @@ impl BorrowChecker {
     /// `GuestSlice` and `GuestStr` structs, which implement `std::ops::Deref` and
     /// `std::ops::DerefMut`. It also enforces that `GuestPtr::read` and `GuestPtr::write` do not
     /// access memory with an outstanding borrow.
-    pub unsafe fn new() -> Self {
+    pub fn new() -> Self {
         BorrowChecker {
             bc: RefCell::new(InnerBorrowChecker::new()),
         }

crates/wiggle/wasmtime/src/lib.rs

@@ -25,7 +25,7 @@ impl WasmtimeGuestMemory {
             // This BorrowChecker construction is a holdover until it is
             // integrated fully with wasmtime:
             // https://github.com/bytecodealliance/wasmtime/issues/1917
-            bc: unsafe { BorrowChecker::new() },
+            bc: BorrowChecker::new(),
         }
     }
 }