Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

padding: ensure PaddingScheme is Send and Sync #215

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

gdesmott
Copy link
Contributor

It's more convenient for users.

Fix #214

@gdesmott
Copy link
Contributor Author

Do we want Sync as well?

@gdesmott
Copy link
Contributor Author

Do we want Sync as well?

Added it as it's free and more convenient.

@gdesmott gdesmott changed the title padding: ensure PaddingScheme is Send padding: ensure PaddingScheme is Send and Sync Oct 20, 2022
@tarcieri
Copy link
Member

This is technically a breaking change, although probably one which won't lead to breakages in practice

@tarcieri tarcieri merged commit a857c8f into RustCrypto:master Oct 24, 2022
@tarcieri tarcieri mentioned this pull request Oct 31, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PaddingScheme being !Send makes things harder
2 participants