Skip to content

Correct the definition of sigval #4637

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

Closed
wants to merge 1 commit into from
Closed

Conversation

yshui
Copy link

@yshui yshui commented Aug 11, 2025

Description

Make the definition of sigval match libc, split from #4620

Sources

https://github.com/bminor/glibc/blob/8543577b04ded6d979ffcc5a818930e4d74d0645/signal/bits/types/__sigval_t.h#L24-L28
https://github.com/bminor/musl/blob/8fd5d031876345e42ae3d11cc07b962f8625bc3b/include/signal.h#L94-L97
https://github.com/torvalds/linux/blob/8f5ae30d69d7543eee0d70083daf4de8fe15d585/include/uapi/asm-generic/siginfo.h#L8-L11

Checklist

  • Relevant tests in libc-test/semver have been updated
  • No placeholder or unstable values like *LAST or *MAX are
    included (see #3131)
  • Tested locally (cd libc-test && cargo test --target mytarget);
    especially relevant for platforms that may not be checked in CI

@yshui
Copy link
Author

yshui commented Aug 11, 2025

oh waht, it has been added already!? #2056

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants