Implement costume changes in target interfaces

Author: adazem009

include/scratchcpp/ispritehandler.h

@@ -18,7 +18,7 @@ class LIBSCRATCHCPP_EXPORT ISpriteHandler
 
         virtual void onCloned(Sprite *clone) = 0;
 
-        virtual void onCostumeChanged(const char *data) = 0;
+        virtual void onCostumeChanged(Costume *costume) = 0;
 
         virtual void onVisibleChanged(bool visible) = 0;
         virtual void onXChanged(double x) = 0;

include/scratchcpp/istagehandler.h

@@ -16,7 +16,7 @@ class LIBSCRATCHCPP_EXPORT IStageHandler
 
         virtual void onStageChanged(Stage *stage) = 0;
 
-        virtual void onCostumeChanged(const char *data) = 0;
+        virtual void onCostumeChanged(Costume *costume) = 0;
 
         virtual void onTempoChanged(int tempo) = 0;
         virtual void onVideoStateChanged(Stage::VideoState videoState) = 0;

include/scratchcpp/stage.h

@@ -29,6 +29,9 @@ class LIBSCRATCHCPP_EXPORT Stage : public Target
         void setInterface(IStageHandler *newInterface);
 
         bool isStage() const override;
+
+        void setCostumeIndex(int newCostumeIndex);
+
         int tempo() const;
         void setTempo(int newTempo);
 

src/scratch/sprite.cpp

@@ -246,6 +246,9 @@ void Sprite::setCostumeIndex(int newCostumeIndex)
     }
 
     Target::setCostumeIndex(newCostumeIndex);
+
+    if (costume && impl->iface)
+        impl->iface->onCostumeChanged(costume.get());
 }
 
 /*! Returns the direction. */

src/scratch/sprite_p.cpp

@@ -31,12 +31,6 @@ void SpritePrivate::removeClone(Sprite *clone)
     }
 }
 
-void SpritePrivate::setCostumeData(const char *data)
-{
-    if (iface)
-        iface->onCostumeChanged(data);
-}
-
 void SpritePrivate::getBoundingRect(Rect *out) const
 {
     assert(out);

src/scratch/sprite_p.h

@@ -17,7 +17,6 @@ struct SpritePrivate
 
         void removeClone(Sprite *clone);
 
-        void setCostumeData(const char *data);
         void getBoundingRect(Rect *out) const;
         void getFencedPosition(double inX, double inY, double *outX, double *outY) const;
 

src/scratch/stage.cpp

@@ -29,6 +29,16 @@ bool Stage::isStage() const
     return true;
 }
 
+/*! Overrides Target#setCostumeIndex(). */
+void Stage::setCostumeIndex(int newCostumeIndex)
+{
+    Target::setCostumeIndex(newCostumeIndex);
+    auto costume = costumeAt(newCostumeIndex);
+
+    if (impl->iface)
+        impl->iface->onCostumeChanged(costume.get());
+}
+
 /*! Returns the tempo. */
 int Stage::tempo() const
 {

src/scratch/stage_p.cpp

@@ -9,9 +9,3 @@ using namespace libscratchcpp;
 StagePrivate::StagePrivate()
 {
 }
-
-void StagePrivate::setCostumeData(const char *data)
-{
-    if (iface)
-        iface->onCostumeChanged(data);
-}

src/scratch/stage_p.h

@@ -12,8 +12,6 @@ struct StagePrivate
         StagePrivate();
         StagePrivate(const StagePrivate &) = delete;
 
-        void setCostumeData(const char *data);
-
         IStageHandler *iface = nullptr;
         int tempo = 60;
         Stage::VideoState videoState = Stage::VideoState::Off;

test/mocks/spritehandlermock.h

@@ -12,7 +12,7 @@ class SpriteHandlerMock : public ISpriteHandler
 
         MOCK_METHOD(void, onCloned, (Sprite *), (override));
 
-        MOCK_METHOD(void, onCostumeChanged, (const char *), (override));
+        MOCK_METHOD(void, onCostumeChanged, (Costume *), (override));
 
         MOCK_METHOD(void, onVisibleChanged, (bool), (override));
         MOCK_METHOD(void, onXChanged, (double), (override));