Skip to content

Commit

Permalink
style(core): fix (hopefully) the weird clippy warning about HOMESCREE…
Browse files Browse the repository at this point in the history
…N_TOIF_X_OFFSET

[no changelog]
  • Loading branch information
matejcik committed Mar 30, 2024
1 parent 379737a commit 1c36c09
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -159,12 +159,6 @@ jobs:
matrix:
model: [T2T1, T2B1]
asan: ${{ fromJSON(needs.param.outputs.asan) }}
# T2B1 fails due to rust clippy error which is hard to reproduce, see discussion here:
# https://github.com/trezor/trezor-firmware/pull/2196
# The problem might be in conflicting versions of clippy, let's see if this helps:
# https://github.com/trezor/trezor-firmware/issues/3337
exclude:
- model: T2B1
env:
TREZOR_MODEL: ${{ matrix.model == 'T2T1' && 'T' || matrix.model == 'T2B1' && 'R' || matrix.model }}
ADDRESS_SANITIZER: ${{ matrix.asan == 'asan' && '1' || '0' }}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ pub const HOMESCREEN_IMAGE_HEIGHT: i16 = HEIGHT;
pub const HOMESCREEN_TOIF_SIZE: i16 = 144;
pub const HOMESCREEN_TOIF_Y_OFFSET: i16 = 27;
pub const HOMESCREEN_TOIF_X_OFFSET: usize =
((WIDTH.saturating_sub(HOMESCREEN_TOIF_SIZE)) / 2) as usize;
((WIDTH as usize).saturating_sub(HOMESCREEN_TOIF_SIZE as usize)) / 2;

const HOMESCREEN_MAX_ICON_SIZE: i16 = 20;
const NOTIFICATION_HEIGHT: i16 = 36;
Expand Down

0 comments on commit 1c36c09

Please sign in to comment.