move around usafe {}

replit · Jan 31, 2024 · 645d292 · 645d292
1 parent cc0b74e
commit 645d292
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 16 deletions.
diff --git a/index.d.ts b/index.d.ts
@@ -4,15 +4,8 @@
 /* auto-generated by NAPI-RS */
 
 export class Pty {
-  fd: number;
-  pid: number;
-  constructor(
-    command: string,
-    args: Array<string>,
-    envs: Record<string, string>,
-    dir: string,
-    size: [cols: number, rows: number],
-    onExit: (err: null | Error, exitCode: number) => void,
-  );
-  resize(size: [cols: number, rows: number]): void;
+  fd: number
+  pid: number
+  constructor(command: string, args: Array<string>, envs: Record<string, string>, dir: string, size: [cols: number, rows: number], onExit: (err: null | Error, exitCode: number) => void)
+  resize(size: [cols: number, rows: number]): void
 }
diff --git a/src/lib.rs b/src/lib.rs
@@ -43,10 +43,10 @@ fn set_controlling_terminal(fd: c_int) -> Result<(), Error> {
 }
 
 #[allow(dead_code)]
-unsafe fn set_nonblocking(fd: c_int) -> Result<(), NAPI_ERROR> {
+fn set_nonblocking(fd: c_int) -> Result<(), NAPI_ERROR> {
   use libc::{fcntl, F_GETFL, F_SETFL, O_NONBLOCK};
 
-  let res = fcntl(fd, F_SETFL, fcntl(fd, F_GETFL, 0) | O_NONBLOCK);
+  let res = unsafe { fcntl(fd, F_SETFL, fcntl(fd, F_GETFL, 0) | O_NONBLOCK) };
 
   if res != 0 {
     return Err(NAPI_ERROR::new(
@@ -131,9 +131,7 @@ impl Pty {
 
     let pid = child.id();
 
-    unsafe {
-      set_nonblocking(fd_controller)?;
-    }
+    set_nonblocking(fd_controller)?;
 
     let file = File::from(pty_pair.controller);
     let fd = file.as_raw_fd();