Do not free cloned asset data

Author: adazem009

include/scratchcpp/asset.h

@@ -38,6 +38,7 @@ class LIBSCRATCHCPP_EXPORT Asset : public Entity
 
     protected:
         virtual void processData(unsigned int size, void *data) { }
+        virtual bool isClone() const { return false; }
 
     private:
         spimpl::unique_impl_ptr<AssetPrivate> impl;

include/scratchcpp/sound.h

@@ -43,6 +43,7 @@ class LIBSCRATCHCPP_EXPORT Sound : public Asset
 
     protected:
         void processData(unsigned int size, void *data) override;
+        virtual bool isClone() const override;
 
     private:
         void stopCloneSounds();

src/scratch/asset.cpp

@@ -17,7 +17,7 @@ Asset::Asset(const std::string &name, const std::string &id, const std::string &
 /*! Destroys Asset. */
 Asset::~Asset()
 {
-    if (impl->data) {
+    if (impl->data && !impl->dataCloned) {
         free(impl->data);
         impl->data = nullptr;
     }
@@ -68,6 +68,7 @@ void Asset::setData(unsigned int size, void *data)
 
     impl->dataSize = size;
     impl->data = data;
+    impl->dataCloned = isClone();
     processData(size, data);
 }
 

src/scratch/asset_p.h

@@ -21,6 +21,7 @@ struct AssetPrivate
         std::string fileName;
         void *data = nullptr;
         unsigned int dataSize = 0;
+        bool dataCloned = false;
         Target *target = nullptr;
 };
 

src/scratch/sound.cpp

@@ -116,6 +116,11 @@ void Sound::processData(unsigned int size, void *data)
         std::cerr << "Failed to load sound " << name() << std::endl;
 }
 
+bool Sound::isClone() const
+{
+    return impl->cloneRoot;
+}
+
 void Sound::stopCloneSounds()
 {
     Target *target = this->target();