define sound once in handleButtons

chee · Mar 18, 2021 · 55a2b1b · 55a2b1b
1 parent af2bc99
commit 55a2b1b
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/bleepbloopmachine/src/main.cc b/bleepbloopmachine/src/main.cc
@@ -446,6 +446,7 @@ class BleepBloopMachine {
     bool b_mode = !ab_mode && held & PAD_B;
     bool start_mode = !ab_mode && !a_mode && !b_mode && held & PAD_START;
     bool none_mode = !ab_mode && !a_mode && !b_mode && !start_mode;
+    auto sound = &selectedWave()->sounds[selectedBlockIndex];
 
     if (b_mode && released) {
       ignoreRelease |= PAD_B;
@@ -499,7 +500,6 @@ class BleepBloopMachine {
         }
       }
       if (selectedMenu1Control == Menu1Selection::env) {
-        auto sound = &selectedWave()->sounds[selectedBlockIndex];
         if (b_mode) {
           if (released & PAD_LEFT) {
             sound->attackDown();
@@ -579,7 +579,7 @@ class BleepBloopMachine {
       break;
     case MenuMode::live:
       if (b_mode) {
-        auto sound = &selectedWave()->sounds[selectedBlockIndex];
+
         if (released & PAD_UP) {
           sound->noteUp();
         }
@@ -609,10 +609,8 @@ class BleepBloopMachine {
         }
 
         if (released == PAD_B) {
-          selectedWave()->sounds[selectedBlockIndex].activate(lastNote,
-                                                              lastOctave);
+          sound->activate(lastNote, lastOctave);
         } else if (released == PAD_A) {
-          auto sound = &selectedWave()->sounds[selectedBlockIndex];
           if (sound->active) {
             // TODO this can be done by assigning it to a variable w/o using &
             clipboard = sound->cut();