2026-05-15 12:29:11

Author: mohanson

src/apu.rs

@@ -258,22 +258,18 @@ impl RegCh4 {
     }
 }
 
-// ────────────────────────────────────────────────────────────────────────────
-// Channel sub-units.
-// ────────────────────────────────────────────────────────────────────────────
-
-/// 6-bit / 8-bit length counter. Returns true from `clock` iff the channel
-/// must be disabled (decrement to zero with length-enable set).
+// Length counter. Returns true from step if the channel must be disabled (decrement to zero with
+// length-enable set).
 struct Lc {
     n: u16,
 }
 
 impl Lc {
-    fn new() -> Self {
+    fn power_up() -> Self {
         Self { n: 0 }
     }
 
-    fn clock(&mut self, en: bool) -> bool {
+    fn step(&mut self, en: bool) -> bool {
         if en && self.n != 0 {
             self.n -= 1;
             self.n == 0
@@ -283,7 +279,7 @@ impl Lc {
     }
 }
 
-/// Volume envelope (CH1/CH2/CH4).
+// Volume envelope (CH1/CH2/CH4).
 struct Env {
     t: Tmr,
     v: u8,
@@ -437,7 +433,7 @@ impl Sq1 {
             r: RegCh1::power_up(),
             on: false,
             tmr: Tmr::power_up(8192),
-            lc: Lc::new(),
+            lc: Lc::power_up(),
             env: Env::new(),
             swp: Swp::new(),
             blip: Blip::power_up(sr),
@@ -486,7 +482,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.clock(true) {
+        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -539,7 +535,7 @@ impl Sq2 {
             r: RegCh2::power_up(),
             on: false,
             tmr: Tmr::power_up(8192),
-            lc: Lc::new(),
+            lc: Lc::power_up(),
             env: Env::new(),
             blip: Blip::power_up(sr),
             idx: 1,
@@ -582,7 +578,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.clock(true) {
+        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -648,7 +644,7 @@ impl Wv {
             r: RegCh3::power_up(),
             on: false,
             tmr: Tmr::power_up(8192),
-            lc: Lc::new(),
+            lc: Lc::power_up(),
             blip: Blip::power_up(sr),
             ram: [0; 16],
             idx: 0,
@@ -696,7 +692,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.clock(true) {
+        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -767,7 +763,7 @@ impl Ns {
             r: RegCh4::power_up(),
             on: false,
             tmr: Tmr::power_up(8),
-            lc: Lc::new(),
+            lc: Lc::power_up(),
             env: Env::new(),
             lfsr: Lfsr::new(),
             blip: Blip::power_up(sr),
@@ -809,7 +805,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.clock(true) {
+        if xtra && !old_en && self.r.r.len_en() && self.lc.step(true) {
             self.on = false;
         }
         if v & 0x80 != 0 {
@@ -1249,16 +1245,16 @@ impl Ticker for Apu {
 
             let s = self.fs.step();
             if Fseq::len(s) {
-                if self.ch1.lc.clock(self.ch1.r.r.len_en()) {
+                if self.ch1.lc.step(self.ch1.r.r.len_en()) {
                     self.ch1.on = false;
                 }
-                if self.ch2.lc.clock(self.ch2.r.r.len_en()) {
+                if self.ch2.lc.step(self.ch2.r.r.len_en()) {
                     self.ch2.on = false;
                 }
-                if self.ch3.lc.clock(self.ch3.r.r.len_en()) {
+                if self.ch3.lc.step(self.ch3.r.r.len_en()) {
                     self.ch3.on = false;
                 }
-                if self.ch4.lc.clock(self.ch4.r.r.len_en()) {
+                if self.ch4.lc.step(self.ch4.r.r.len_en()) {
                     self.ch4.on = false;
                 }
             }