Skip to content

Commit 6976745

Browse files
authored
Apply Clippy 0.1.85's new suggestions (#44)
1 parent 947f4a4 commit 6976745

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

src/array_decoder/list.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ impl ArrayBatchDecoder for ListArrayDecoder {
7878
// Fetch child array as one Array with total_length elements
7979
let child_array = self.inner.next_batch(total_length as usize, None)?;
8080
let offsets = OffsetBuffer::from_lengths(lengths.into_iter().map(|l| l as usize));
81-
let null_buffer = present.map(NullBuffer::from);
81+
let null_buffer = present;
8282

8383
let array = ListArray::try_new(self.field.clone(), offsets, child_array, null_buffer)
8484
.context(ArrowSnafu)?;

src/array_decoder/struct_decoder.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ impl ArrayBatchDecoder for StructArrayDecoder {
7070
.map(|child| child.next_batch(batch_size, present.as_ref()))
7171
.collect::<Result<Vec<_>>>()?;
7272

73-
let null_buffer = present.map(NullBuffer::from);
73+
let null_buffer = present;
7474
let array = StructArray::try_new(self.fields.clone(), child_arrays, null_buffer)
7575
.context(ArrowSnafu)?;
7676
let array = Arc::new(array);

src/encoding/integer/rle_v2/delta.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ fn derive_delta_header(delta_width: usize, run_length: usize) -> [u8; 2] {
175175
let encoded_length_high_bit = (run_length >> 8) as u8;
176176
let encoded_length_low_bits = (run_length & 0xFF) as u8;
177177

178-
let header1 = EncodingType::Delta.to_header() | delta_width << 1 | encoded_length_high_bit;
178+
let header1 = EncodingType::Delta.to_header() | (delta_width << 1) | encoded_length_high_bit;
179179
let header2 = encoded_length_low_bits;
180180

181181
[header1, header2]

src/encoding/integer/rle_v2/patched_base.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -256,10 +256,10 @@ pub fn write_patched_base(
256256
let encoded_patch_gap_width = patch_gap_width - 1;
257257

258258
let header1 =
259-
EncodingType::PatchedBase.to_header() | encoded_bit_width << 1 | encoded_length_high_bit;
259+
EncodingType::PatchedBase.to_header() | (encoded_bit_width << 1) | encoded_length_high_bit;
260260
let header2 = encoded_length_low_bits;
261-
let header3 = (encoded_base_width as u8) << 5 | encoded_patch_bits_width as u8;
262-
let header4 = (encoded_patch_gap_width as u8) << 5 | patches.len() as u8;
261+
let header3 = ((encoded_base_width as u8) << 5) | encoded_patch_bits_width as u8;
262+
let header4 = ((encoded_patch_gap_width as u8) << 5) | patches.len() as u8;
263263
writer.put_slice(&[header1, header2, header3, header4]);
264264

265265
// Write out base value as big endian bytes

0 commit comments

Comments
 (0)