-
Notifications
You must be signed in to change notification settings - Fork 191
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
support u128 #201
support u128 #201
Conversation
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
bors: r+ |
201: support u128 r=Dylan-DPC a=kinggoesgaming **I'm submitting a ...** - [ ] bug fix - [x] feature enhancement - [ ] deprecation or removal - [ ] refactor # Description * features introduced: `u128`, `nightly` * rust channel needed: `nightly` * dependency introduced: `byteorder` This introduced implementation for `u128` in the from of a `Uuid::from_u128` and `impl From` # Motivation `Uuid`s are 128 bits in size and rust natively provides a `u128` integer type. This # Tests `u128_support::test_from_u128()` introduced and passes # Related Issue(s) N/A Co-authored-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
/// To create a [`Uuid`] from `u128`s, you need `u128` feature enabled for this crate. | ||
/// | ||
/// [`Uuid`]: ../struct.Uuid.html | ||
#[inline(always)] |
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.
Usually inline(always)
isn't ideal. A simple inline
(without the always
) would be better. Did you have a case where this method wasn't being inlined?
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.
actually that is a good point... I think I will remove in a future PR that does misc cleanup
* Fix buffer not being modified * Add unit tests for v5 buffer * Handle old node versions for test
I'm submitting a ...
Description
u128
,nightly
nightly
byteorder
This introduced implementation for
u128
in the from of aUuid::from_u128
andimpl From
Motivation
Uuid
s are 128 bits in size and rust natively provides au128
integer type. ThisTests
u128_support::test_from_u128()
introduced and passesRelated Issue(s)
N/A