chore: remove various warnings

src/apu.rs

@@ -131,10 +131,11 @@ impl Apu {
 
         self.div_prev = Some(bit_5);
 
-        if let Some(ref mut prod) = self.prod {
         if self.sample_counter >= SM83_CLOCK_SPEED {
             self.sample_counter %= SM83_CLOCK_SPEED;
 
+            if let Some(ref mut prod) = self.prod {
+                if prod.two_available() {
                     // Sample the APU
                     let ch1_amplitude = self.ch1.amplitude();
                     let ch1_left = self.ctrl.output.ch1_left() as u8 as f32 * ch1_amplitude;
@@ -161,18 +162,12 @@ impl Apu {
                 }
             }
         }
+    }
 
     pub fn set_producer(&mut self, prod: SampleProducer<f32>) {
         self.prod = Some(prod);
     }
 
-    pub(crate) fn is_full(&self) -> bool {
-        match self.prod.as_ref() {
-            Some(prod) => prod.is_full(),
-            None => false,
-        }
-    }
-
     /// 0xFF26 | NR52 - Sound On/Off
     pub(crate) fn set_status(&mut self, byte: u8) {
         self.ctrl.enabled = (byte >> 7) & 0x01 == 0x01;

src/apu/gen.rs

@@ -3,7 +3,7 @@ use rtrb::{Consumer, Producer, PushError, RingBuffer};
 
 pub(crate) const SAMPLE_RATE: u32 = 48000; // Hz
 const CHANNEL_COUNT: usize = 2;
-const AUDIO_BUFFER_LEN: usize = 4096;
+const BUFFER_CAPACITY: usize = 4096 * CHANNEL_COUNT; // # of samples * the # of channels
 
 pub struct AudioSPSC<T> {
     inner: RingBuffer<T>,
@@ -12,7 +12,7 @@ pub struct AudioSPSC<T> {
 impl<T> Default for AudioSPSC<T> {
     fn default() -> Self {
         Self {
-            inner: RingBuffer::new(AUDIO_BUFFER_LEN * CHANNEL_COUNT),
+            inner: RingBuffer::new(BUFFER_CAPACITY),
         }
     }
 }
@@ -43,8 +43,9 @@ impl<T> SampleProducer<T> {
         self.inner.push(value)
     }
 
-    pub(crate) fn is_full(&self) -> bool {
+    #[inline]
-        self.inner.is_full()
+    pub(crate) fn two_available(&self) -> bool {
+        self.inner.slots() > 2
     }
 }
 

src/bus.rs

@@ -66,10 +66,6 @@ impl Bus {
         self.cartridge.as_ref()?.title()
     }
 
-    pub(crate) fn apu(&self) -> &Apu {
-        &self.apu
-    }
-
     pub(crate) fn clock(&mut self) {
         self.ppu.clock();
         self.timer.clock();

src/cpu.rs

@@ -117,16 +117,9 @@ impl Cpu {
             }
         };
 
-        // TODO: With how we currently handle audio
-        // this --while being correct-- incurs a performance penalty
-        // as our emu is audio-bound.
-        let mut elapsed = 0x00;
         let pending: u32 = cycles.into();
-        while elapsed < pending {
+        for _ in 0..pending {
-            if !self.bus.apu().is_full() {
             self.bus.clock();
-                elapsed += 1;
-            }
         }
 
         self.handle_interrupts();

src/main.rs

@@ -112,7 +112,7 @@ fn main() -> Result<()> {
 
             if cycle_count >= gb::emu::CYCLES_IN_FRAME {
                 // Draw Frame
-                cycle_count = Cycle::new(0);
+                cycle_count %= gb::emu::CYCLES_IN_FRAME;
 
                 gb::emu::draw(game_boy.ppu(), pixels.get_frame());
                 window.request_redraw();