Skip to content
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

Berid of most (all?) warnings #1206

Merged
merged 3 commits into from
May 22, 2020

Conversation

nabijaczleweli
Copy link
Member

@nabijaczleweli nabijaczleweli commented Apr 19, 2020

Mainly the vomit from ExtendedColorType in each and every build, but tested with no features, one feature at a time, and all features, all of which have just the two warnings fixed by #1205.

src/color.rs Outdated Show resolved Hide resolved
Copy link
Member

@HeroicKatora HeroicKatora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After adding the documentation to the color types, it definitely feels like it was the better choice. The wording made me realize that previously the bit-depth was not clear if it was per-channel or per-pixel.

@HeroicKatora HeroicKatora merged commit 8a4de73 into image-rs:master May 22, 2020
@HeroicKatora
Copy link
Member

HeroicKatora commented May 22, 2020

Looks like the crate itself is free of warnings. Very nice 🎉

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

Successfully merging this pull request may close these issues.

3 participants