Skip to content

Commit

Permalink
zircon: the type of zx_handle_t is now unsigned
Browse files Browse the repository at this point in the history
This is a kernel ABI change that landed today. I noticed some other ABI
issues and have left a note to cleanup once they are better defined.
  • Loading branch information
raggi committed Oct 4, 2017
1 parent 417ffc9 commit 17a2a76
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/libstd/sys/unix/process/zircon.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,16 @@ use u64;

use libc::{c_int, c_void};

pub type zx_handle_t = i32;
pub type zx_handle_t = u32;
pub type zx_vaddr_t = usize;
pub type zx_rights_t = u32;
pub type zx_status_t = i32;

// TODO(raggi): zx_size_t was removed from Zircon. various syscall API docs use size_t here, but
// don't define that well at the ABI level yet, as the C spec definition of size_t isn't what is
// meant. In the future Zirocn will define size_t more strictly for it's ABI. At that time,
// zx_size_t should be removed here, and replaced with an appropriately sized type with a
// sufficiently strict definition.
pub type zx_size_t = usize;

pub const ZX_HANDLE_INVALID: zx_handle_t = 0;
Expand Down

0 comments on commit 17a2a76

Please sign in to comment.