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

Init a BInt from a [UInt8] #56

Merged
merged 4 commits into from
Aug 25, 2021
Merged

Init a BInt from a [UInt8] #56

merged 4 commits into from
Aug 25, 2021

Conversation

nathanfallet
Copy link
Contributor

@nathanfallet nathanfallet commented Aug 25, 2021

Adding the ability to initialize a BInt from bytes using a [UInt8].
I will use this feature for a RSA implementation (as we are working from bytes and big numbers)

Checklist:

  • Initialize from [UInt8]
  • Export to [UInt8]

Linked to #52 and krzyzanowskim/CryptoSwift#887

@nathanfallet
Copy link
Contributor Author

I need to add another commit (I think tomorrow) for the second item of my checklist before this PR get merged.

@twodayslate
Copy link
Collaborator

Sounds good - thanks!

@nathanfallet
Copy link
Contributor Author

@twodayslate My PR is ready, you can review it and merge it if everything is ok

@twodayslate twodayslate merged commit 0ec7214 into mkrd:master Aug 25, 2021
@nathanfallet nathanfallet deleted the init branch August 25, 2021 22:18
@nathanfallet
Copy link
Contributor Author

@twodayslate Is it possible to release a 2.1.2 so that we can use this addition in production?

@twodayslate
Copy link
Collaborator

@nathanfallet done - thanks for the reminder and thank you for the contribution!

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