Fixed bug introduced in trunk r211.1.9, where Anduril's goodnight mode didn't work sometimes.

The problem was that goodnight's EV_enter was executing before battcheck's loop() finished.
This meant battcheck's exit code would turn the emitter off right after goodnight's init set it to the desired level.

1 files changed, 6 insertions, 2 deletions

diff --git a/spaghetti-monster/fsm-events.c b/spaghetti-monster/fsm-events.c
index aa5c3d6..c71e822 100644
--- a/spaghetti-monster/fsm-events.c
+++ b/spaghetti-monster/fsm-events.c
@@ -160,11 +160,15 @@ uint8_t nice_delay_ms(uint16_t ms) {
         _delay_loop_2(BOGOMIPS*98/100);
         #endif  // ifdef USE_DYNAMIC_UNDERCLOCKING
 
-        process_emissions();
         if ((nice_delay_interrupt) || (old_state != current_state)) {
-            //nice_delay_interrupt = 0;
             return 0;  // state changed; abort
         }
+        // handle events only afterward, so that any collapsed delays will
+        // finish running the UI's loop() code before taking any further actions
+        // (this helps make sure code runs in the correct order)
+        // (otherwise, a new state's EV_enter runs before the old state's
+        //  loop() has finished, and things can get weird)
+        process_emissions();
     }
     return 1;
 }