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.

Sign up for GitHub

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

Jump to bottom

try_copy! macro #53

Merged
merged 0 commits into from
May 1, 2019
Merged

try_copy! macro #53

merged 0 commits into from
May 1, 2019

Conversation

Enet4
Copy link
Collaborator

@Enet4 Enet4 commented Apr 29, 2019 •
edited by nabijaczleweli
Loading

This is an attempt to make the crate a bit easier to work with. Basically, one can grab the outcome of a transmutation and make a transmuted copy if the operation failed due to certain constraints.

  • add try_copy! and try_copy_unchecked! to root
  • add copy, copy_unchecked and without_src to Error
  • fix documentation mistakes in type parameter names
  • test macro in aligned_vec tests

nabijaczleweli
nabijaczleweli requested changes Apr 30, 2019
View reviewed changes
Copy link
Owner

@nabijaczleweli nabijaczleweli left a comment

Choose a reason for hiding this comment

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

Delightful!

src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
tests/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@nabijaczleweli nabijaczleweli merged commit d126067 into nabijaczleweli:master May 1, 2019
@Enet4 Enet4 deleted the imp/try_copy branch May 1, 2019 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nabijaczleweli nabijaczleweli nabijaczleweli requested changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Enet4 @nabijaczleweli