anduril: move candle mode to loop()

Bug #1802873 reported by Selene ToyKeeper on 2018-11-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Flashlight Firmware Repository
Low
Unassigned

Bug Description

Candle mode's logic is implemented inside its event handler. This works, but it means it's locked at 62 fps. I'd like to move the logic to the inside of loop() instead, to clean up an already-cluttered event handler and make it easier to adjust parameters like the frame rate.

On a related note, it might also be a good idea to break out parts of loop() into their own functions, if it can be done without increasing the ROM size. It has gotten pretty cluttered too.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers