We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
OrdF64
NonNan
It'd be better to model OrdF64 as NonNan because it brings panic (or error) closer to the cause of the issue making debugging easier. Potential inspiration: https://docs.rs/ordered-float/3.0.0/ordered_float/struct.NotNan.html
The change should be simple: make the field private and constructor panics or returns Result.
Result
The text was updated successfully, but these errors were encountered:
concept ACK
Sorry, something went wrong.
Indeed the fuzzer found the test case "or(0@TRIVIAL,0@TRIVIAL)" that is panicking
@RCasatta I assume that means returning Result (probably using TryFrom) is the right approach?
TryFrom
Yes, I think so.
No branches or pull requests
It'd be better to model
OrdF64
asNonNan
because it brings panic (or error) closer to the cause of the issue making debugging easier. Potential inspiration: https://docs.rs/ordered-float/3.0.0/ordered_float/struct.NotNan.htmlThe change should be simple: make the field private and constructor panics or returns
Result
.The text was updated successfully, but these errors were encountered: