Skip to content

polyval: use ManuallyDrop unions; MSRV 1.49+#113

Merged
tarcieri merged 1 commit intomasterfrom
polyval/manually-drop-unions
Dec 31, 2020
Merged

polyval: use ManuallyDrop unions; MSRV 1.49+#113
tarcieri merged 1 commit intomasterfrom
polyval/manually-drop-unions

Conversation

@tarcieri
Copy link
Member

Using ManuallyDrop for the fields of unions (allowing them to be non-Copy types) was stabilized in Rust 1.49.

Using `ManuallyDrop` for the fields of `union`s (allowing them to be
non-Copy types) was stabilized in Rust 1.49.
@tarcieri tarcieri force-pushed the polyval/manually-drop-unions branch from 253c4cb to 563fa52 Compare December 31, 2020 16:21
@codecov-io
Copy link

Codecov Report

Merging #113 (563fa52) into master (e75d01f) will decrease coverage by 0.90%.
The diff coverage is 7.69%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage   50.04%   49.14%   -0.91%     
==========================================
  Files          14       14              
  Lines        1091     1107      +16     
==========================================
- Hits          546      544       -2     
- Misses        545      563      +18     
Impacted Files Coverage Δ
polyval/src/backend/clmul.rs 75.92% <0.00%> (-4.47%) ⬇️
polyval/src/backend/soft32.rs 0.00% <0.00%> (ø)
polyval/src/backend/soft64.rs 0.00% <0.00%> (ø)
polyval/src/backend/autodetect.rs 44.00% <25.00%> (+1.69%) ⬆️
poly1305/src/backend/avx2.rs 88.88% <0.00%> (-3.71%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e75d01f...563fa52. Read the comment docs.

@tarcieri tarcieri merged commit f26bb48 into master Dec 31, 2020
@tarcieri tarcieri deleted the polyval/manually-drop-unions branch December 31, 2020 16:24
tarcieri added a commit that referenced this pull request Dec 31, 2020
Using `ManuallyDrop` for the fields of `union`s (allowing them to be
non-Copy types) was stabilized in Rust 1.49.
tarcieri added a commit to RustCrypto/AEADs that referenced this pull request Dec 31, 2020
The `aes` crate is a key component of several crates in this repo, and
just had an MSRV breaking change in RustCrypto/block-ciphers#216.

This commit bumps the MSRV of all crates in this repo which use the
`aes` crate to MSRV 1.49+.

In the case of `aes-gcm` and `aes-gcm-siv` specifically, the changes in
RustCrypto/universal-hashes#113 bumped the MSRV of `ghash` and `polyval`
to 1.49+ as well, so there are multiple motivations for this new MSRV.
tarcieri added a commit to RustCrypto/AEADs that referenced this pull request Dec 31, 2020
The `aes` crate is a key component of several crates in this repo, and
just had an MSRV breaking change in RustCrypto/block-ciphers#216.

This commit bumps the MSRV of all crates in this repo which use the
`aes` crate to MSRV 1.49+.

In the case of `aes-gcm` and `aes-gcm-siv` specifically, the changes in
RustCrypto/universal-hashes#113 bumped the MSRV of `ghash` and `polyval`
to 1.49+ as well, so there are multiple motivations for this new MSRV.
tarcieri added a commit to RustCrypto/AEADs that referenced this pull request Dec 31, 2020
The `aes` crate is a key component of several crates in this repo, and
just had an MSRV breaking change in RustCrypto/block-ciphers#216.

This commit bumps the MSRV of all crates in this repo which use the
`aes` crate to MSRV 1.49+.

In the case of `aes-gcm` and `aes-gcm-siv` specifically, the changes in
RustCrypto/universal-hashes#113 bumped the MSRV of `ghash` and `polyval`
to 1.49+ as well, so there are multiple motivations for this new MSRV.
tarcieri added a commit to RustCrypto/AEADs that referenced this pull request Dec 31, 2020
The `aes` crate is a key component of several crates in this repo, and
just had an MSRV breaking change in RustCrypto/block-ciphers#216.

This commit bumps the MSRV of all crates in this repo which use the
`aes` crate to MSRV 1.49+.

In the case of `aes-gcm` and `aes-gcm-siv` specifically, the changes in
RustCrypto/universal-hashes#113 bumped the MSRV of `ghash` and `polyval`
to 1.49+ as well, so there are multiple motivations for this new MSRV.
@tarcieri tarcieri mentioned this pull request Apr 29, 2021
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