Skip to content

crypto.ecdsa: preparation step to migrate to the new api. #23595

Merged
spytheman merged 1 commit into
vlang:masterfrom
blackshirt:ext_ecdsa
Jan 27, 2025
Merged

crypto.ecdsa: preparation step to migrate to the new api. #23595
spytheman merged 1 commit into
vlang:masterfrom
blackshirt:ext_ecdsa

Conversation

@blackshirt

@blackshirt blackshirt commented Jan 27, 2025

Copy link
Copy Markdown
Contributor

This preparations PR was related to talk to migrate crypto.ecdsa module to use the new high level API. This PR contains simple changes in the form:

  • Split out C declarations into the new file ecdsa.c.v to align with the habits of the rest of standard modules that wraps C lib.
  • Adds a new field of high level EVP_PKEY opaque to the PrivateKey and PublicKey part.

Its doesn't add functionality, just preparation step.

Thanks

…the C wrapper to the new file and add the new opaque to the PrivateKey and PublicKey declarations.
@huly-for-github

Copy link
Copy Markdown

Connected to Huly®: V_0.6-22021

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work.

@spytheman spytheman merged commit 1eaa7c8 into vlang:master Jan 27, 2025
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