diff --git a/src/cpu.rs b/src/cpu.rs
index f557f595da36993d34197d24c110a7d9ee0f1a31..fe7f9c1b1f11fcd5f3aeb3e0de8414a2db810532 100644
--- a/src/cpu.rs
+++ b/src/cpu.rs
@@ -381,7 +381,7 @@ impl Cpu {
pub fn read_u16(&mut self) -> u16 {
let byte1 = self.read_u8();
let byte2 = self.read_u8();
-
+
byte1 as u16 | ((byte2 as u16) << 8)
}
@@ -406,7 +406,6 @@ impl Cpu {
#[inline(always)]
pub fn pop_word(&mut self) -> u16 {
-
self.pop_byte() as u16 | ((self.pop_byte() as u16) << 8)
}
diff --git a/src/gb.rs b/src/gb.rs
index 01f57fd0363272fbaef900dfb6f2c74290ffa427..73af0a2529fa590ccf9efaf6c26cfcd070c12e4a 100644
--- a/src/gb.rs
+++ b/src/gb.rs
@@ -13,7 +13,7 @@ use crate::{
use wasm_bindgen::prelude::*;
#[cfg(feature = "wasm")]
-use crate::ppu::{Pixel, Palette};
+use crate::ppu::{Palette, Pixel};
#[cfg(feature = "wasm")]
use std::{
@@ -206,7 +206,7 @@ impl GameBoy {
self.cpu.timer()
}
- pub fn frame_buffer(&mut self) -> &Box<[u8; FRAME_BUFFER_SIZE]> {
+ pub fn frame_buffer(&mut self) -> &[u8; FRAME_BUFFER_SIZE] {
&(self.ppu().frame_buffer)
}
@@ -292,3 +292,9 @@ pub fn hook_impl(info: &PanicInfo) {
let message = info.to_string();
panic(message.as_str());
}
+
+impl Default for GameBoy {
+ fn default() -> Self {
+ Self::new()
+ }
+}
diff --git a/src/inst.rs b/src/inst.rs
index 7d9891b22799c6c75c4f876a233f799f09f9adee..c0a16aa2a41b1070c32c29b084b7f00e2059389b 100644
--- a/src/inst.rs
+++ b/src/inst.rs
@@ -1,6 +1,6 @@
use crate::cpu::Cpu;
-pub const INSTRUCTIONS: [(fn(&mut Cpu), u8, &str); 256] = [
+pub const INSTRUCTIONS: [Instruction; 256] = [
// 0x0 opcodes
(nop, 4, "NOP"),
(ld_bc_u16, 12, "LD BC, u16"),
@@ -275,7 +275,7 @@ pub const INSTRUCTIONS: [(fn(&mut Cpu), u8, &str); 256] = [
(rst_38h, 16, "RST 38h"),
];
-pub const EXTENDED: [(fn(&mut Cpu), u8, &str); 256] = [
+pub const EXTENDED: [Instruction; 256] = [
// 0x0 opcodes
(rlc_b, 8, "RLC B"),
(rlc_c, 8, "RLC C"),
@@ -550,6 +550,8 @@ pub const EXTENDED: [(fn(&mut Cpu), u8, &str); 256] = [
(set_7_a, 8, "SET 7, A"),
];
+pub type Instruction = (fn(&mut Cpu), u8, &'static str);
+
fn nop(_cpu: &mut Cpu) {}
fn illegal(_cpu: &mut Cpu) {
@@ -1040,9 +1042,7 @@ fn ccf(cpu: &mut Cpu) {
cpu.set_carry(!cpu.get_carry());
}
-fn ld_b_b(cpu: &mut Cpu) {
- cpu.b = cpu.b;
-}
+fn ld_b_b(_cpu: &mut Cpu) {}
fn ld_b_c(cpu: &mut Cpu) {
cpu.b = cpu.c;
@@ -1077,9 +1077,7 @@ fn ld_c_b(cpu: &mut Cpu) {
cpu.c = cpu.b;
}
-fn ld_c_c(cpu: &mut Cpu) {
- cpu.c = cpu.c;
-}
+fn ld_c_c(_cpu: &mut Cpu) {}
fn ld_c_d(cpu: &mut Cpu) {
cpu.c = cpu.d;
@@ -1114,9 +1112,7 @@ fn ld_d_c(cpu: &mut Cpu) {
cpu.d = cpu.c;
}
-fn ld_d_d(cpu: &mut Cpu) {
- cpu.d = cpu.d;
-}
+fn ld_d_d(_cpu: &mut Cpu) {}
fn ld_d_e(cpu: &mut Cpu) {
cpu.d = cpu.e;
@@ -1151,9 +1147,7 @@ fn ld_e_d(cpu: &mut Cpu) {
cpu.e = cpu.d;
}
-fn ld_e_e(cpu: &mut Cpu) {
- cpu.e = cpu.e;
-}
+fn ld_e_e(_cpu: &mut Cpu) {}
fn ld_e_h(cpu: &mut Cpu) {
cpu.e = cpu.h;
@@ -1188,9 +1182,7 @@ fn ld_h_e(cpu: &mut Cpu) {
cpu.h = cpu.e;
}
-fn ld_h_h(cpu: &mut Cpu) {
- cpu.h = cpu.h;
-}
+fn ld_h_h(_cpu: &mut Cpu) {}
fn ld_h_l(cpu: &mut Cpu) {
cpu.h = cpu.l;
@@ -1225,9 +1217,7 @@ fn ld_l_h(cpu: &mut Cpu) {
cpu.l = cpu.h;
}
-fn ld_l_l(cpu: &mut Cpu) {
- cpu.l = cpu.l;
-}
+fn ld_l_l(_cpu: &mut Cpu) {}
fn ld_l_mhl(cpu: &mut Cpu) {
let byte = cpu.mmu.read(cpu.hl());
@@ -1299,9 +1289,7 @@ fn ld_a_mhl(cpu: &mut Cpu) {
cpu.a = byte;
}
-fn ld_a_a(cpu: &mut Cpu) {
- cpu.a = cpu.a;
-}
+fn ld_a_a(_cpu: &mut Cpu) {}
fn add_a_b(cpu: &mut Cpu) {
cpu.a = add_set_flags(cpu, cpu.a, cpu.b);
diff --git a/src/mmu.rs b/src/mmu.rs
index c18e1f7c8e5fc6021b9f3a148c9f08e98830be82..2847a46fb298b9ac1d9c98938ccfd713e9146e41 100644
--- a/src/mmu.rs
+++ b/src/mmu.rs
@@ -230,9 +230,9 @@ impl Mmu {
}
}
- pub fn write_many(&mut self, addr: u16, data: &Vec<u8>) {
- for index in 0..data.len() {
- self.write(addr + index as u16, data[index])
+ pub fn write_many(&mut self, addr: u16, data: &[u8]) {
+ for (index, byte) in data.iter().enumerate() {
+ self.write(addr + index as u16, *byte)
}
}
diff --git a/src/pad.rs b/src/pad.rs
index 84c833a158465586a4b741bbd99b4e467afe186f..0f2298ffe9350cdf3bdcd1aa0f7adaaa2a364962 100644
--- a/src/pad.rs
+++ b/src/pad.rs
@@ -53,21 +53,20 @@ impl Pad {
pub fn read(&mut self, addr: u16) -> u8 {
match addr & 0x00ff {
0x0000 => {
- let mut value;
- match self.selection {
+ let mut value = match self.selection {
PadSelection::Action => {
- value = if self.a { 0x00 } else { 0x01 }
+ (if self.a { 0x00 } else { 0x01 }
| if self.b { 0x00 } else { 0x02 }
| if self.select { 0x00 } else { 0x04 }
- | if self.start { 0x00 } else { 0x08 }
+ | if self.start { 0x00 } else { 0x08 })
}
PadSelection::Direction => {
- value = if self.right { 0x00 } else { 0x01 }
+ (if self.right { 0x00 } else { 0x01 }
| if self.left { 0x00 } else { 0x02 }
| if self.up { 0x00 } else { 0x04 }
- | if self.down { 0x00 } else { 0x08 }
+ | if self.down { 0x00 } else { 0x08 })
}
- }
+ };
value |= if self.selection == PadSelection::Direction {
0x10
} else {
@@ -141,3 +140,9 @@ impl Pad {
self.set_int_pad(false);
}
}
+
+impl Default for Pad {
+ fn default() -> Self {
+ Self::new()
+ }
+}
diff --git a/src/ppu.rs b/src/ppu.rs
index c871a4521f72d727a9bfb0d218da69be6deb2ca6..bccbc8e3b59b18563ce5bb91ca76d46e5b81bc95 100644
--- a/src/ppu.rs
+++ b/src/ppu.rs
@@ -882,12 +882,11 @@ impl Ppu {
tile = &self.tiles[obj.tile as usize];
}
- let tile_row: &[u8];
- if obj.yflip {
- tile_row = tile.get_row((7 - tile_offset) as usize);
+ let tile_row = if obj.yflip {
+ tile.get_row((7 - tile_offset) as usize)
} else {
- tile_row = tile.get_row((tile_offset) as usize);
- }
+ tile.get_row((tile_offset) as usize)
+ };
for x in 0..TILE_WIDTH {
let is_contained =
@@ -936,3 +935,9 @@ impl Ppu {
|| self.stat_hblank && self.mode == PpuMode::HBlank
}
}
+
+impl Default for Ppu {
+ fn default() -> Self {
+ Self::new()
+ }
+}
diff --git a/src/rom.rs b/src/rom.rs
index b01458eb489a74f8ef4a408fa2b3d6a1f114c9a9..26a09286e6616beadfc39c397cc657e442f934ed 100644
--- a/src/rom.rs
+++ b/src/rom.rs
@@ -412,6 +412,12 @@ impl Cartridge {
}
}
+impl Default for Cartridge {
+ fn default() -> Self {
+ Self::new()
+ }
+}
+
impl Display for Cartridge {
fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
write!(
diff --git a/src/timer.rs b/src/timer.rs
index dfe137211d236f80990eae6c5a7c7c55e95e6e2c..e1fb4a1a397180b52fde04e14336b6be66cc60fc 100644
--- a/src/timer.rs
+++ b/src/timer.rs
@@ -100,3 +100,9 @@ impl Timer {
self.set_int_tima(false);
}
}
+
+impl Default for Timer {
+ fn default() -> Self {
+ Self::new()
+ }
+}