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
Update image and approx crate dependency #104
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two in one! Nice! Other than what I think is a logic mistake, it looks really good. Thanks!
palette/src/equality.rs
Outdated
fn default_max_ulps() -> u32 { | ||
T::default_max_ulps() | ||
fn abs_diff_ne(&self, other: &Self, epsilon: T::Epsilon) -> bool { | ||
$( self.$element.abs_diff_ne(&other.$element, epsilon) )&&+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these should be delimited with ||
, since it would be the inverse of abs_diff_eq
. Same for the other *_ne
implementations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right that should be ||
, looks it was implemented wrong previously as well because the *_ne
were using &&
. I just adapted that by accident. Fixed it now if I didnt miss anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, good thing it was discovered then.
I think that was all. Great! bors r+ |
Build succeeded |
Closes #101 and closes #100. I accidentally put both pull requests together.