2026-05-15 14:05:50

Author: mohanson

src/apu.rs

@@ -269,13 +269,12 @@ impl Lc {
         Self { n: 0 }
     }
 
-    fn step(&mut self, en: bool) -> bool {
-        if en && self.n != 0 {
+    fn step(&mut self, reg: &Reg) -> bool {
+        if reg.len_en() && self.n != 0 {
             self.n -= 1;
-            self.n == 0
-        } else {
-            false
+            return self.n == 0;
         }
+        return false;
     }
 }
 
@@ -482,7 +481,7 @@ impl Sq1 {
         let old_en = self.r.r.len_en();
         self.r.r.n[4] = v;
         self.tmr.p = self.r.period();
-        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
+        if xtra && !old_en && self.lc.step(&self.r.r) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -578,7 +577,7 @@ impl Sq2 {
         let old_en = self.r.r.len_en();
         self.r.r.n[4] = v;
         self.tmr.p = self.r.period();
-        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
+        if xtra && !old_en && self.lc.step(&self.r.r) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -692,7 +691,7 @@ impl Wv {
         let old_en = self.r.r.len_en();
         self.r.r.n[4] = v;
         self.tmr.p = self.r.period();
-        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
+        if xtra && !old_en && self.lc.step(&self.r.r) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -805,7 +804,7 @@ impl Ns {
     fn write_nrx4(&mut self, v: u8, xtra: bool) {
         let old_en = self.r.r.len_en();
         self.r.r.n[4] = v;
-        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
+        if xtra && !old_en && self.lc.step(&self.r.r) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -1245,16 +1244,16 @@ impl Ticker for Apu {
 
             let s = self.fs.step();
             if Fseq::len(s) {
-                if self.ch1.lc.step(self.ch1.r.r.len_en()) {
+                if self.ch1.lc.step(&self.ch1.r.r) {
                     self.ch1.on = false;
                 }
-                if self.ch2.lc.step(self.ch2.r.r.len_en()) {
+                if self.ch2.lc.step(&self.ch2.r.r) {
                     self.ch2.on = false;
                 }
-                if self.ch3.lc.step(self.ch3.r.r.len_en()) {
+                if self.ch3.lc.step(&self.ch3.r.r) {
                     self.ch3.on = false;
                 }
-                if self.ch4.lc.step(self.ch4.r.r.len_en()) {
+                if self.ch4.lc.step(&self.ch4.r.r) {
                     self.ch4.on = false;
                 }
             }