update BoxDraw to allow bidirectional connection between ImageFloatBox and individual coordinates.

Author: jw098

SerialPrograms/Source/NintendoSwitch/DevPrograms/BoxDraw.cpp

@@ -33,7 +33,7 @@ BoxDraw::BoxDraw()
     , Y("<b>Y Coordinate:</b>", LockMode::UNLOCK_WHILE_RUNNING, 0.3, 0.0, 1.0)
     , WIDTH("<b>Width:</b>", LockMode::UNLOCK_WHILE_RUNNING, 0.4, 0.0, 1.0)
     , HEIGHT("<b>Height:</b>", LockMode::UNLOCK_WHILE_RUNNING, 0.4, 0.0, 1.0)
-    , BOX_COORDINATES(false, "ImageFloatBox coordinates", LockMode::LOCK_WHILE_RUNNING, "0.3, 0.3, 0.4, 0.4", "0.3, 0.3, 0.4, 0.4")
+    , BOX_COORDINATES(false, "ImageFloatBox coordinates", LockMode::UNLOCK_WHILE_RUNNING, "0.3, 0.3, 0.4, 0.4", "0.3, 0.3, 0.4, 0.4")
 {
     PA_ADD_OPTION(X);
     PA_ADD_OPTION(Y);
@@ -59,23 +59,23 @@ class BoxDraw::DrawnBox : public ConfigOption::Listener, public VideoOverlay::Mo
             m_parent.Y.add_listener(*this);
             m_parent.WIDTH.add_listener(*this);
             m_parent.HEIGHT.add_listener(*this);
-            // m_parent.BOX_COORDINATES.add_listener(*this);
+            m_parent.BOX_COORDINATES.add_listener(*this);
             overlay.add_listener(*this);
         }catch (...){
             detach();
             throw;
         }
     }
     virtual void on_config_value_changed(void* object) override{
-        std::lock_guard<std::mutex> lg(m_lock);
+        // std::lock_guard<std::mutex> lg(m_lock);
         m_overlay_set.clear();
         m_overlay_set.add(COLOR_RED, {m_parent.X, m_parent.Y, m_parent.WIDTH, m_parent.HEIGHT});
         if (object == &m_parent.X || object == &m_parent.Y || object == &m_parent.WIDTH || object == &m_parent.HEIGHT){
             m_parent.update_box_coordinates();
         }
-        // else if(object == &m_parent.BOX_COORDINATES){
-        //     // m_parent.update_individual_coordinates();    
-        // }
+        else if(object == &m_parent.BOX_COORDINATES){
+            m_parent.update_individual_coordinates();    
+        }
 
 
     }
@@ -112,7 +112,7 @@ class BoxDraw::DrawnBox : public ConfigOption::Listener, public VideoOverlay::Mo
         m_parent.Y.set(yl);
         m_parent.WIDTH.set(xh - xl);
         m_parent.HEIGHT.set(yh - yl);
-        m_parent.update_box_coordinates();
+        // m_parent.update_box_coordinates();
     }
 
 private:
@@ -122,6 +122,7 @@ class BoxDraw::DrawnBox : public ConfigOption::Listener, public VideoOverlay::Mo
         m_parent.Y.remove_listener(*this);
         m_parent.WIDTH.remove_listener(*this);
         m_parent.HEIGHT.remove_listener(*this);
+        m_parent.BOX_COORDINATES.remove_listener(*this);
     }
 
 private:
@@ -134,7 +135,8 @@ class BoxDraw::DrawnBox : public ConfigOption::Listener, public VideoOverlay::Mo
 };
 
 void BoxDraw::update_box_coordinates(){
-    BOX_COORDINATES.set(std::to_string(X) + ", " + std::to_string(Y) + ", " + std::to_string(WIDTH) + ", " + std::to_string(HEIGHT));
+    std::string box_coord_string = std::to_string(X) + ", " + std::to_string(Y) + ", " + std::to_string(WIDTH) + ", " + std::to_string(HEIGHT);
+    BOX_COORDINATES.set(box_coord_string);
 }
 
 std::vector<std::string> split(const std::string& str, const std::string& delimiter) {
@@ -167,11 +169,11 @@ void BoxDraw::update_individual_coordinates(){
     double width_coord = std::stod(width_string);
     double height_coord = std::stod(height_string);
 
-    cout << box_coord_string << endl;
-    cout << std::to_string(x_coord) << endl;
-    cout << std::to_string(y_coord) << endl;
-    cout << std::to_string(width_coord) << endl;
-    cout << std::to_string(height_coord) << endl;
+    // cout << box_coord_string << endl;
+    // cout << std::to_string(x_coord) << endl;
+    // cout << std::to_string(y_coord) << endl;
+    // cout << std::to_string(width_coord) << endl;
+    // cout << std::to_string(height_coord) << endl;
 
     X.set(x_coord);
     Y.set(y_coord);