Deal with freezing issue in models without PSRAM

Author: mongonta0716

src/Avatar.cpp

@@ -36,11 +36,14 @@ void facialLoop(void *args) {
   unsigned long last_saccade_millis = 0;
   unsigned long last_blink_millis = 0;
   bool eye_open = true;
+  float vertical = 0.0f;
+  float horizontal = 0.0f;
+  float breath = 0.0f;
   while (avatar->isDrawing()) {
 
     if ((millis() - last_saccade_millis) > saccade_interval) {
-      float vertical = rand_r(&seed) / (RAND_MAX / 2.0) - 1;
-      float horizontal = rand_r(&seed) / (RAND_MAX / 2.0) - 1;
+      vertical = rand_r(&seed) / (RAND_MAX / 2.0) - 1;
+      horizontal = rand_r(&seed) / (RAND_MAX / 2.0) - 1;
       avatar->setGaze(vertical, horizontal);
       saccade_interval = 500 + 100 * random(20);
       last_saccade_millis = millis();
@@ -58,8 +61,8 @@ void facialLoop(void *args) {
       last_blink_millis = millis();
     }
     c = (c + 1) % 100;
-    float f = sin(c * 2 * PI / 100.0);
-    avatar->setBreath(f);
+    breath = sin(c * 2 * PI / 100.0);
+    avatar->setBreath(breath);
     vTaskDelay(33);
   }
   vTaskDelete(NULL);
@@ -83,6 +86,10 @@ Avatar::Avatar(Face *face)
       colorDepth{1},
       batteryIconStatus{BatteryIconStatus::invisible}{}
 
+Avatar::~Avatar() {
+  delete face;
+}
+
 void Avatar::setFace(Face *face) { this->face = face; }
 
 Face *Avatar::getFace() const { return face; }
@@ -159,7 +166,9 @@ void Avatar::draw() {
 bool Avatar::isDrawing() { return _isDrawing; }
 
 void Avatar::setExpression(Expression expression) {
+  suspend();
   this->expression = expression;
+  resume();
 }
 
 Expression Avatar::getExpression() {

src/Avatar.h

@@ -31,7 +31,7 @@ class Avatar {
  public:
   Avatar();
   explicit Avatar(Face *face);
-  ~Avatar() = default;
+  ~Avatar();
   Avatar(const Avatar &other) = default;
   Avatar &operator=(const Avatar &other) = default;
   Face *getFace() const;
@@ -61,6 +61,7 @@ class Avatar {
   void resume();
   void setBatteryIcon(bool iconStatus);
   void setBatteryStatus(bool isCharging, int32_t batteryLevel);
+  void resetDraw();
 };
 
 

src/Face.cpp

@@ -5,9 +5,6 @@
 #include "Face.h"
 
 namespace m5avatar {
-Balloon b;
-Effect h;
-BatteryIcon battery;
 BoundingRect br;
 
 Face::Face()
@@ -54,6 +51,9 @@ Face::~Face() {
   delete eyeblowLPos;
   delete sprite;
   delete boundingRect;
+  delete b;
+  delete h;
+  delete battery;
 }
 
 void Face::setMouth(Drawable *mouth) { this->mouth = mouth; }
@@ -106,9 +106,9 @@ void Face::draw(DrawContext *ctx) {
   eyeblowL->draw(sprite, rect, ctx);
 
   // TODO(meganetaaan): make balloons and effects selectable
-  b.draw(sprite, br, ctx);
-  h.draw(sprite, br, ctx);
-  battery.draw(sprite, br, ctx);
+  b->draw(sprite, br, ctx);
+  h->draw(sprite, br, ctx);
+  battery->draw(sprite, br, ctx);
   // drawAccessory(sprite, position, ctx);
 
   // TODO(meganetaaan): rethink responsibility for transform function

src/Face.h

@@ -30,6 +30,9 @@ class Face {
   BoundingRect *boundingRect;
   M5Canvas *sprite;
   M5Canvas *tmpSprite;
+  Balloon *b;
+  Effect *h;
+  BatteryIcon *battery;
 
  public:
   // constructor