Skip to content

Commit 516ca3d

Browse files
committed
clean up
Signed-off-by: Connor Tsui <[email protected]>
1 parent ad43042 commit 516ca3d

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

encodings/fastlanes/src/bitpacking/array/unpack_iter.rs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -182,19 +182,23 @@ impl<T: PhysicalPType, S: UnpackStrategy<T>> UnpackedChunks<T, S> {
182182
if let Some(initial) = self.initial() {
183183
local_idx = initial.len();
184184

185+
// TODO(connor): use `maybe_uninit_write_slice` feature when it gets stabilized.
186+
// https://github.com/rust-lang/rust/issues/79995
185187
// SAFETY: &[T] and &[MaybeUninit<T>] have the same layout.
186-
let uninit_initial: &[MaybeUninit<T>] = unsafe { mem::transmute(initial) };
187-
output[..uninit_initial.len()].copy_from_slice(uninit_initial);
188+
let init_initial: &[MaybeUninit<T>] = unsafe { mem::transmute(initial) };
189+
output[..local_idx].copy_from_slice(init_initial);
188190
}
189191

190192
// Handle full chunks
191193
local_idx = self.decode_full_chunks_into_at(output, local_idx);
192194

193195
// Handle trailing partial chunk if present
194196
if let Some(trailer) = self.trailer() {
197+
// TODO(connor): use `maybe_uninit_write_slice` feature when it gets stabilized.
198+
// https://github.com/rust-lang/rust/issues/79995
195199
// SAFETY: &[T] and &[MaybeUninit<T>] have the same layout.
196-
let uninit_trailer: &[MaybeUninit<T>] = unsafe { mem::transmute(trailer) };
197-
output[local_idx..][..uninit_trailer.len()].copy_from_slice(uninit_trailer);
200+
let init_trailer: &[MaybeUninit<T>] = unsafe { mem::transmute(trailer) };
201+
output[local_idx..][..init_trailer.len()].copy_from_slice(init_trailer);
198202
}
199203
}
200204

0 commit comments

Comments
 (0)