Skip to content

sapemu: Implement shifts on direct page #683

sapemu: Implement shifts on direct page

sapemu: Implement shifts on direct page #683

Triggered via push September 10, 2024 11:51
Status Failure
Total duration 1m 39s
Artifacts

rust.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 21 warnings
this argument is a mutable reference, but not used mutably: sapemu/src/smp/ops.rs#L2193
error: this argument is a mutable reference, but not used mutably --> sapemu/src/smp/ops.rs:2193:10 | 2193 | memory: &mut Memory, | ^^^^^^^^^^^ help: consider changing to: `&Memory` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_ref_mut = note: `-D clippy::needless-pass-by-ref-mut` implied by `-D clippy::nursery` = help: to override `-D clippy::nursery` add `#[allow(clippy::needless_pass_by_ref_mut)]`
item in documentation is missing backticks: sapemu/src/test.rs#L201
error: item in documentation is missing backticks --> sapemu/src/test.rs:201:47 | 201 | /// Tests that are not run due to issues with SingleStepTest's disregard of hardware properties. | ^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown = note: `-D clippy::doc-markdown` implied by `-D clippy::pedantic` = help: to override `-D clippy::pedantic` add `#[allow(clippy::doc_markdown)]` help: try | 201 | /// Tests that are not run due to issues with `SingleStepTest`'s disregard of hardware properties. | ~~~~~~~~~~~~~~~~
this could be a `const fn`: sapemu/src/smp.rs#L429
error: this could be a `const fn` --> sapemu/src/smp.rs:429:2 | 429 | / fn carry(&self) -> bool { 430 | | self.psw.contains(ProgramStatusWord::Carry) 431 | | } | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_const_for_fn help: make the function `const` | 429 | const fn carry(&self) -> bool { | +++++
casting to the same type is unnecessary (`u16` -> `u16`): sapemu/src/smp/ops.rs#L2091
error: casting to the same type is unnecessary (`u16` -> `u16`) --> sapemu/src/smp/ops.rs:2091:50 | 2091 | MicroArchAction::Continue(state.with_address2(address as u16)) | ^^^^^^^^^^^^^^ help: try: `address` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast = note: `-D clippy::unnecessary-cast` implied by `-D clippy::all` = help: to override `-D clippy::all` add `#[allow(clippy::unnecessary_cast)]`
this could be a `const fn`: sapemu/src/smp/ops.rs#L1696
error: this could be a `const fn` --> sapemu/src/smp/ops.rs:1696:1 | 1696 | / fn increment_wrap_within_page(address: u16) -> u16 { 1697 | | let low_byte_increment = (address & 0xff).wrapping_add(1) & 0xff; 1698 | | address & 0xff00 | low_byte_increment 1699 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_const_for_fn = note: `-D clippy::missing-const-for-fn` implied by `-D clippy::nursery` = help: to override `-D clippy::nursery` add `#[allow(clippy::missing_const_for_fn)]` help: make the function `const` | 1696 | const fn increment_wrap_within_page(address: u16) -> u16 { | +++++
clippy
Clippy had exited with the 101 exit code
fmt
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
fmt
The following actions use a deprecated Node.js version and will be forced to run on node20: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
fmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
fmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
fmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
fmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
fmt
The process '/usr/bin/git' failed with exit code 128
build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
build
The process '/usr/bin/git' failed with exit code 128
clippy
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1.0.7. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
clippy
The following actions use a deprecated Node.js version and will be forced to run on node20: actions-rs/toolchain@v1, actions-rs/clippy-check@v1.0.7. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
clippy
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
clippy
The process '/usr/bin/git' failed with exit code 128