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

Updating secp256k1 and supporting full no-std features #114

Closed
elichai opened this issue May 28, 2019 · 1 comment · Fixed by #125
Closed

Updating secp256k1 and supporting full no-std features #114

elichai opened this issue May 28, 2019 · 1 comment · Fixed by #125

Comments

@elichai
Copy link
Member

elichai commented May 28, 2019

Now that both bitcoin-core/secp256k1#566 and bitcoin-core/secp256k1#595 Are merged,

I would like to update the code in depend to secp256k1 master and start working on integrating both of these features into the library here.

I'm not sure how the preallocation will look like i'll know more after i'll dive into the C code and try and see what's possible in rust, but if there are no bounds to that memory at all the user of this library might need to provide the buffer.

i'll start working on this this weekend. just a heads up in case anyone else thought of doing this :)

Any comments/ideas are welcome :)

@elichai elichai changed the title Updating secp256k1 Updating secp256k1 and supporting full no-std features May 28, 2019
@elichai
Copy link
Member Author

elichai commented May 28, 2019

What do you guys think about using https://github.com/rust-lang/rust-bindgen to generate the rust definitions?

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 a pull request may close this issue.

1 participant