rust-lang/rust
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Remove redundant Aligner

Browse Source 
The `Aligner` struct seems to be unnecessary.
Previously noted by @arthurprs https://github.com/rust-lang/rust/pull/44963#discussion_r145340754
Reddit discussion: https://www.reddit.com/r/rust/comments/pfvvz2/aligner_and_cachealigned/
Playground: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=fa7ca554922755f9d1b62b017d785c6f
This commit is contained in:
Tony Yang
2021-10-26 11:34:03 +01:00
parent 29b1248025
commit f54663767d
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
library/std/src/sync/mpsc/cache_aligned.rs
View File

@@ -2,10 +2,7 @@ use crate::ops::{Deref, DerefMut};
#[derive(Copy, Clone, Default, PartialEq, Eq, PartialOrd, Ord, Hash)] #[derive(Copy, Clone, Default, PartialEq, Eq, PartialOrd, Ord, Hash)]
#[repr(align(64))] #[repr(align(64))]
pub(super) struct Aligner; pub(super) struct CacheAligned<T>(pub T);
#[derive(Copy, Clone, Default, PartialEq, Eq, PartialOrd, Ord, Hash)]
pub(super) struct CacheAligned<T>(pub T, pub Aligner);
impl<T> Deref for CacheAligned<T> { impl<T> Deref for CacheAligned<T> {
type Target = T; type Target = T;
@@ -22,6 +19,6 @@ impl<T> DerefMut for CacheAligned<T> {
impl<T> CacheAligned<T> { impl<T> CacheAligned<T> {
pub(super) fn new(t: T) -> Self { pub(super) fn new(t: T) -> Self {
CacheAligned(t, Aligner) CacheAligned(t)
} }
} }