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

[std::span] Tweakable_Block_Cipher #3872

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

reneme
Copy link
Collaborator

@reneme reneme commented Jan 4, 2024

Pull Request Dependencies

Description

Introduces a private polymorphy base Tweakable_Block_Cipher::set_tweak_value() and adds a public ::set_tweak() overload taking a std::span<>.

@coveralls
Copy link

coveralls commented Jan 4, 2024

Coverage Status

coverage: 92.083% (-0.008%) from 92.091%
when pulling 06f6347 on Rohde-Schwarz:span/tweakable_block_cipher
into 38a0b56 on randombit:master.

This demotes Block_Cipher::encrypt_n/decrypt_n to top-level methods
and introduces new (private) virtual methods (encrypt/decrypt_blocks)
that use std::span for the in/out buffers. Also, this adapts all block
cipher implementations in the library to use the new API.
@reneme reneme force-pushed the span/tweakable_block_cipher branch from 825007d to 06f6347 Compare March 25, 2024 15:37
@reneme reneme modified the milestones: Botan 3.4.0, Botan 3.5.0 Apr 8, 2024
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.

None yet

2 participants