Added clippy and fmt to CI and fixes clippy issues

Signed-off-by: Amjad Alsharafi <[email protected]>
Amjad50 · Oct 23, 2024 · dbec9f3 · dbec9f3
1 parent 0b78c71
commit dbec9f3
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 11 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -33,6 +33,11 @@ jobs:
 
     - uses: actions/checkout@v4
 
+    - name: Format check
+      run: cargo fmt --all -- --check
+    - name: Clippy check
+      run: cargo clippy --all --all-targets --all-features -- -D warnings
+
     # breaks for now since Cargo.lock is in a new version (4)
     # - uses: actions-rs/audit-check@v1
     #   with:

diff --git a/plastic_core/src/cartridge/mappers/mapper10.rs b/plastic_core/src/cartridge/mappers/mapper10.rs
@@ -65,10 +65,6 @@ pub struct Mapper10 {
     ///         +- Nametable mirroring (0: vertical; 1: horizontal)
     mirroring_vertical: bool,
 
-    /// ($C000-$DFFE, even)
-    /// the value to reload `irq_counter` when it reaches zero or when asked
-    /// to be reloaded from `($C001-$DFFF, odd)`
-
     /// is using CHR RAM?
     is_chr_ram: bool,
 

diff --git a/plastic_core/src/cartridge/mappers/mapper9.rs b/plastic_core/src/cartridge/mappers/mapper9.rs
@@ -65,10 +65,6 @@ pub struct Mapper9 {
     ///         +- Nametable mirroring (0: vertical; 1: horizontal)
     mirroring_vertical: bool,
 
-    /// ($C000-$DFFE, even)
-    /// the value to reload `irq_counter` when it reaches zero or when asked
-    /// to be reloaded from `($C001-$DFFF, odd)`
-
     /// is using CHR RAM?
     is_chr_ram: bool,
 

diff --git a/plastic_core/src/cpu6502/mod.rs b/plastic_core/src/cpu6502/mod.rs
@@ -1443,7 +1443,7 @@ struct WrapperReader<'a, R: Read> {
     pub inner: &'a mut R,
 }
 
-impl<'a, R: Read> Read for WrapperReader<'a, R> {
+impl<R: Read> Read for WrapperReader<'_, R> {
     fn read(&mut self, buf: &mut [u8]) -> std::io::Result<usize> {
         self.inner.read(buf)
     }

diff --git a/plastic_core/src/misc.rs b/plastic_core/src/misc.rs
@@ -8,6 +8,12 @@ pub struct MovingAverage {
     sum: f64,
 }
 
+impl Default for MovingAverage {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl MovingAverage {
     pub fn new() -> Self {
         Self {

diff --git a/plastic_core/src/ppu2c02/sprite.rs b/plastic_core/src/ppu2c02/sprite.rs
@@ -65,9 +65,8 @@ impl Sprite {
 
     /// for 8x8:
     /// use the byte normally as index into the pattern table
-
+    ///
     /// for 8x16:
-
     /// 76543210
     /// ||||||||
     /// |||||||+- Bank ($0000 or $1000) of tiles