-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Open
Description
edit: This is now more a meta issue to discuss an improved context API, see the discussion below.
I'm arguably late to the party but I believe we should either talk about this now (before we do a release) or never:
Taking a step back, I believe that even given the problems with C, upstream's API is not optimal here. A better API would be to ask the user to provide entropy already in
_context_create
, or pass NULL explicitly to opt out.
Originally posted by @real-or-random in rust-bitcoin/rust-secp256k1#225 (comment)
Note that this does not need to be a breaking change, we could for example add a new function and deprecate secp256k1_context_create
.
elichai
Metadata
Metadata
Assignees
Labels
No labels