Skip to content

Commit 83d9398

Browse files
committed
Address clippy lints
Fix the following: - clippy::manual_div_ceil - clippy::needless_lifetimes - clippy::precedence - clippy::repr_packed_without_abi Signed-off-by: Tim Crawford <[email protected]>
1 parent c69c7bf commit 83d9398

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

src/app/cmos.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ impl CmosOptionTable {
5858
let hi = u16::from(self.cmos.read(Self::CHECKSUM_OFFSET));
5959
let lo = u16::from(self.cmos.read(Self::CHECKSUM_OFFSET + 1));
6060

61-
hi << 8 | lo
61+
(hi << 8) | lo
6262
}
6363

6464
/// Write the checksum to the CMOS option table.

src/app/ec.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ impl<T: Timeout> SpiLegacy<T> {
404404

405405
unsafe fn read(&mut self, data: &mut [u8]) -> core::result::Result<(), ectool::Error> {
406406
let block_size = self.block_size();
407-
let blocks = (data.len() + block_size - 1) / block_size;
407+
let blocks = data.len().div_ceil(block_size);
408408
for block in 0..blocks {
409409
unsafe {
410410
self.pmc_cmd(0x03)?;
@@ -427,7 +427,7 @@ impl<T: Timeout> SpiLegacy<T> {
427427

428428
unsafe fn write(&mut self, data: &[u8]) -> core::result::Result<(), ectool::Error> {
429429
let block_size = self.block_size();
430-
let blocks = (data.len() + block_size - 1) / block_size;
430+
let blocks = data.len().div_ceil(block_size);
431431
for block in 0..blocks {
432432
unsafe {
433433
self.pmc_cmd(0x02)?;

src/app/pci.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ use std::prelude::*;
66
use std::uefi::guid;
77

88
#[allow(dead_code)]
9-
#[repr(packed)]
9+
#[repr(C, packed)]
1010
struct Rsdp {
1111
signature: [u8; 8], // b"RSD PTR "
1212
chksum: u8,
@@ -21,7 +21,7 @@ struct Rsdp {
2121
}
2222

2323
#[allow(dead_code)]
24-
#[repr(packed)]
24+
#[repr(C, packed)]
2525
struct SdtHeader {
2626
signature: [u8; 4],
2727
length: u32,

src/display.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ impl<'a> ScaledDisplay<'a> {
137137
}
138138
}
139139

140-
impl<'a> Renderer for ScaledDisplay<'a> {
140+
impl Renderer for ScaledDisplay<'_> {
141141
fn width(&self) -> u32 {
142142
self.display.width() / self.scale
143143
}

src/image/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ pub struct ImageRoi<'a> {
1717
image: &'a Image,
1818
}
1919

20-
impl<'a> ImageRoi<'a> {
20+
impl ImageRoi<'_> {
2121
/// Draw the ROI on a window
2222
pub fn draw<R: Renderer>(&self, renderer: &mut R, x: i32, mut y: i32) {
2323
let stride = self.image.w;

0 commit comments

Comments
 (0)