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

Separate sRGB and gamma encoded RGB from the Rgb type #24

Merged
merged 6 commits into from
Jan 24, 2016
Merged

Conversation

Ogeon
Copy link
Owner

@Ogeon Ogeon commented Jan 24, 2016

This closes #7 by adding separate types for sRGB and gamma encoded RGB. Rgb becomes a purely linear variant of sRGB and the types for encoded sRGB and gamma RGB can be found in a newly added pixel module. The RgbPixel trait has also been moved to the pixel module.

The linear_* prefix has also been dropped from the Rgb constructors and some bugs from #18 were fixed as a bonus.

This is a breaking change, since it moves, removes and renames a lot of functions, and the RgbPixel trait is no longer available in the crate root.

@Ogeon
Copy link
Owner Author

Ogeon commented Jan 24, 2016

There were some talk somewhere about supporting AdobeRGB, but that can be added later. I'll call this one done.

@homu r+

@homu
Copy link
Contributor

homu commented Jan 24, 2016

📌 Commit babb5ee has been approved by Ogeon

@homu
Copy link
Contributor

homu commented Jan 24, 2016

⚡ Test exempted - status

@homu homu merged commit babb5ee into master Jan 24, 2016
homu added a commit that referenced this pull request Jan 24, 2016
Separate sRGB and gamma encoded RGB from the Rgb type

This closes #7 by adding separate types for sRGB and gamma encoded RGB. `Rgb` becomes a purely linear variant of sRGB and the types for encoded sRGB and gamma RGB can be found in a newly added `pixel` module. The `RgbPixel` trait has also been moved to the `pixel` module.

The `linear_*` prefix has also been dropped from the `Rgb` constructors and some bugs from #18 were fixed as a bonus.

This is a breaking change, since it moves, removes and renames a lot of functions, and the `RgbPixel` trait is no longer available in the crate root.
@Ogeon Ogeon deleted the rgb_separation branch January 27, 2016 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Separation of the RGB types
2 participants