Skip to content

Updated crates.io #502

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

Merged
merged 1 commit into from
Jan 5, 2016
Merged

Updated crates.io #502

merged 1 commit into from
Jan 5, 2016

Conversation

bvssvni
Copy link
Contributor

@bvssvni bvssvni commented Jan 5, 2016

  • Bumped to 0.6.1
  • Moved methods to GenericImage impl

@bvssvni
Copy link
Contributor Author

bvssvni commented Jan 5, 2016

The rand crate fails on Rust nightly rust-lang/rust#30713.

rustc 1.7.0-nightly (d5e229057 2016-01-04)

@tafia
Copy link
Contributor

tafia commented Jan 5, 2016

I think this is not a breaking change as these fn already had a default
impl through GenericImage.
On 5 Jan 2016 21:56, "Sven Nilsen" [email protected] wrote:

  • Bumped to 0.7.0

Breaking changes:

  • Added ImageBuffer::unsafe_get_pixel
  • Added ImageBuffer::unsafe_put_pixel

You can view, comment on, or merge this pull request online at:

#502
Commit Summary

  • Updated crates.io

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#502.

@bvssvni
Copy link
Contributor Author

bvssvni commented Jan 5, 2016

@tafia Perhaps the new methods should be moved here?

@bvssvni
Copy link
Contributor Author

bvssvni commented Jan 5, 2016

@tafia I moved the methods to the GenericImage impl. Changed the version to 0.6.1. Had to change from &P to P when returning from unsafe_get_pixel.

@bvssvni
Copy link
Contributor Author

bvssvni commented Jan 5, 2016

Merging.

bvssvni added a commit that referenced this pull request Jan 5, 2016
@bvssvni bvssvni merged commit b287376 into image-rs:master Jan 5, 2016
@bvssvni bvssvni deleted the upd branch January 5, 2016 20:03
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.

2 participants