-
Notifications
You must be signed in to change notification settings - Fork 682
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
SigSet Incorrect implementation of Eq and PartialEq #1749
Labels
Comments
I can work on fixing this, I assume it is preferable to manually implement |
Yes, I think it would be preferable to implement PartialEq rather than not to. |
FYI: I previously reported this to libc |
@germag are you still planning to fix this? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
signal::SigSet
derivesEq
andPartialEq
, I assume becauselibc::sigset_t
does it too, but in linux,sigfillset(3)
only initializes the first 64 bits, leaving the rest uninitialized, so twosignal::SigSets
can be different even though both have the same set of signalsThe text was updated successfully, but these errors were encountered: