diff options
| author | Thomas Zander <thomas.e.zander@googlemail.com> | 2021-05-29 14:19:40 +0200 |
|---|---|---|
| committer | Thomas Zander <thomas.e.zander@googlemail.com> | 2021-05-29 14:19:40 +0200 |
| commit | 09d12f8cbd27efa99db47dc412a09872565ecd86 (patch) | |
| tree | 460018db822d96d319422941e6c1642b6ef52992 | |
| parent | docs(changelog): Optional session name in attach #542 (diff) | |
| download | sl0ppij-09d12f8cbd27efa99db47dc412a09872565ecd86.tar.gz sl0ppij-09d12f8cbd27efa99db47dc412a09872565ecd86.tar.xz sl0ppij-09d12f8cbd27efa99db47dc412a09872565ecd86.zip | |
Fix build on platforms with TIOCGWINSZ / ioctl() integer type mismatch.
| -rw-r--r-- | zellij-client/src/os_input_output.rs | 8 | ||||
| -rw-r--r-- | zellij-server/src/os_input_output.rs | 6 |
2 files changed, 12 insertions, 2 deletions
diff --git a/zellij-client/src/os_input_output.rs b/zellij-client/src/os_input_output.rs index 0606ae37..405422ab 100644 --- a/zellij-client/src/os_input_output.rs +++ b/zellij-client/src/os_input_output.rs @@ -45,7 +45,13 @@ pub(crate) fn get_terminal_size_using_fd(fd: RawFd) -> PositionAndSize { ws_ypixel: 0, }; - unsafe { ioctl(fd, TIOCGWINSZ, &mut winsize) }; + // TIOCGWINSZ is an u32, but the second argument to ioctl is u64 on + // some platforms. When checked on Linux, clippy will complain about + // useless conversion. + #[allow(clippy::useless_conversion)] + unsafe { + ioctl(fd, TIOCGWINSZ.into(), &mut winsize) + }; PositionAndSize::from(winsize) } diff --git a/zellij-server/src/os_input_output.rs b/zellij-server/src/os_input_output.rs index 4cda73ed..91a228e7 100644 --- a/zellij-server/src/os_input_output.rs +++ b/zellij-server/src/os_input_output.rs @@ -41,7 +41,11 @@ pub(crate) fn set_terminal_size_using_fd(fd: RawFd, columns: u16, rows: u16) { ws_xpixel: 0, ws_ypixel: 0, }; - unsafe { ioctl(fd, TIOCSWINSZ, &winsize) }; + // TIOCGWINSZ is an u32, but the second argument to ioctl is u64 on + // some platforms. When checked on Linux, clippy will complain about + // useless conversion. + #[allow(clippy::useless_conversion)] + unsafe { ioctl(fd, TIOCSWINSZ.into(), &winsize) }; } /// Handle some signals for the child process. This will loop until the child |
