support animations without fps and frames

Author: timhendriks93

src/BlenderServoAnimation.cpp

@@ -4,10 +4,12 @@
 
 using namespace BlenderServoAnimation;
 
+Animation::Animation() {
+}
+
 Animation::Animation(byte fps, int frames) {
   this->frameMillis = Animation::SECOND_IN_MILLIS / fps;
   this->frames = frames;
-  this->mode = Animation::MODE_DEFAULT;
 }
 
 void Animation::addServo(Servo servo) {
@@ -39,7 +41,9 @@ void Animation::run(unsigned long currentMillis) {
 }
 
 void Animation::handlePlayMode() {
-  if (this->currentMillis - this->lastMillis < this->frameMillis) {
+  bool isNewFrame = this->currentMillis - this->lastMillis >= this->frameMillis;
+
+  if (!isNewFrame || this->frames == 0) {
     return;
   }
 

src/BlenderServoAnimation.h

@@ -11,12 +11,12 @@ class Animation {
   static const int MAX_SERVO_COUNT = 256;
   static const int SECOND_IN_MILLIS = 1000;
 
-  byte frameMillis;
-  byte mode;
-  byte stopStepDelay;
+  byte frameMillis = 0;
+  byte stopStepDelay = 20;
+  byte mode = MODE_DEFAULT;
 
   int frame = 0;
-  int frames;
+  int frames = 0;
 
   unsigned long currentMillis;
   unsigned long lastMillis;
@@ -38,6 +38,8 @@ class Animation {
   static const byte MODE_STOP = 3;
   static const byte MODE_LIVE = 4;
 
+  Animation();
+
   Animation(byte fps, int frames);
 
   void addServo(Servo servo);

test/test_animation/test_animation.cpp

@@ -123,7 +123,7 @@ void test_stop(void) {
 }
 
 void test_live(void) {
-  Animation animation(FPS, 5);
+  Animation animation;
   SerialMock mock;
   Servo servos[] = {
       Servo(0, positionsA, move),