2026-05-15 15:02:17

Author: mohanson

src/apu.rs

@@ -211,7 +211,7 @@ struct Env {
 }
 
 impl Env {
-    fn new() -> Self {
+    fn power_up() -> Self {
         Self { t: Tmr::power_up(8), v: 0 }
     }
 
@@ -222,11 +222,11 @@ impl Env {
         self.v = reg.vol_init();
     }
 
-    fn clock(&mut self, p: u8, add: bool) {
-        if p == 0 || self.t.step(1) == 0 {
+    fn step(&mut self, reg: &Reg) {
+        if reg.env_p() == 0 || self.t.step(1) == 0 {
             return;
         }
-        let n = if add { self.v.wrapping_add(1) } else { self.v.wrapping_sub(1) };
+        let n = if reg.env_add() { self.v.wrapping_add(1) } else { self.v.wrapping_sub(1) };
         if n <= 15 {
             self.v = n;
         }
@@ -361,7 +361,7 @@ impl Sq1 {
             on: false,
             tmr: Tmr::power_up(8192),
             lc: Lc::power_up(),
-            env: Env::new(),
+            env: Env::power_up(),
             swp: Swp::new(),
             blip: Blip::power_up(sr),
             idx: 1,
@@ -465,7 +465,7 @@ impl Sq2 {
             on: false,
             tmr: Tmr::power_up(8192),
             lc: Lc::power_up(),
-            env: Env::new(),
+            env: Env::power_up(),
             blip: Blip::power_up(sr),
             idx: 1,
         }
@@ -693,7 +693,7 @@ impl Ns {
             on: false,
             tmr: Tmr::power_up(8),
             lc: Lc::power_up(),
-            env: Env::new(),
+            env: Env::power_up(),
             lfsr: Lfsr::new(),
             blip: Blip::power_up(sr),
         }
@@ -1189,13 +1189,13 @@ impl Ticker for Apu {
             }
             if Fseq::env(s) {
                 if self.ch1.on {
-                    self.ch1.env.clock(self.ch1.r.env_p(), self.ch1.r.env_add());
+                    self.ch1.env.step(&self.ch1.r);
                 }
                 if self.ch2.on {
-                    self.ch2.env.clock(self.ch2.r.env_p(), self.ch2.r.env_add());
+                    self.ch2.env.step(&self.ch2.r);
                 }
                 if self.ch4.on {
-                    self.ch4.env.clock(self.ch4.r.env_p(), self.ch4.r.env_add());
+                    self.ch4.env.step(&self.ch4.r);
                 }
             }
             if Fseq::swp(s) && self.ch1.on {