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

BigInt wrapper and local types #135

Closed
wants to merge 1 commit into from
Closed

Conversation

austinabell
Copy link
Contributor

Will leave as draft until agreed upon. Another option is to just alias the Bigint types through this crate without wrapping it just to keep consistent version and allowing it to be more easily changed if custom functionality is needed in future.

All of this is optional but seeing interactions needed in the near future, for example Cbor encoding, let me know if you guys want this.

I only changed usage in the vm package as well for this, since no one else is modifying, and can change the other references once agreed upon and probably in a seperate PR

@austinabell
Copy link
Contributor Author

Just going to close this for now, if we need to add our own functionality on this type can just use these changes as a template to reimplement.

This was referenced Jan 14, 2020
@austinabell austinabell deleted the austin/math/bigint branch January 16, 2020 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant