-
Notifications
You must be signed in to change notification settings - Fork 144
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
Add u128 feature for complete u128 compatibility #57
Conversation
This seems to only be used in the formatting pieces, and there could we just use |
Implemented as suggested. The change does require absolute paths for custom types and type aliases. |
Oh right sorry I forgot about that :(, that's actually a pretty crucial reason for this feature. In that case I guess we'll have to stick with an unstable feature |
I will revert to my original PR. Any comments or suggestions? |
Looks like this may still be the requires-absolute-paths version? |
Thanks! |
#61 reimplements this in a different way that neither requires absolute paths nor a cfg to opt into u128 compatibility. |
Added a u128 feature with a feature gated bitflags width type for use in the macro. Enables functionality like debug formatting of flags that exceed 64 bits. #56