Skip to content

Auto merge of #516 - binarycrusader:master, r=alexcrichton #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 10, 2017

Conversation

binarycrusader
Copy link
Owner

Correct solaris libc definitions:

  • pthread_t is defined as uint_t, so must be c_uint, not uintptr_t, just
    as pthread_key_t is already defined
  • fd_set is defined as long, so must be i32/i64 based on
    target_pointer_width; this also fixes an indirect endianness issue
    encountered on sparc
  • FD_SETSIZE should be defined as 65536 when target_pointer_width = 64

Fixes rust-lang#515

Correct solaris libc definitions:

* pthread_t is defined as uint_t, so must be c_uint, not uintptr_t, just
  as pthread_key_t is already defined
* fd_set is defined as long, so must be i32/i64 based on
  target_pointer_width; this also fixes an indirect endianness issue
  encountered on sparc
* FD_SETSIZE should be defined as 65536 when target_pointer_width = 64

Fixes #515
@binarycrusader binarycrusader merged commit c9548a5 into binarycrusader:master Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

some libc solaris definitions are not correct
2 participants