Target: Use references when returning assets

Author: adazem009

include/scratchcpp/target.h

@@ -53,12 +53,12 @@ class LIBSCRATCHCPP_EXPORT Target
 
         const std::vector<Costume> &costumes() const;
         int addCostume(const Costume &costume);
-        Costume costumeAt(int index) const;
+        const Costume &costumeAt(int index) const;
         int findCostume(const std::string &costumeName) const;
 
         const std::vector<Sound> &sounds() const;
         int addSound(const Sound &sound);
-        Sound soundAt(int index) const;
+        const Sound &soundAt(int index) const;
         int findSound(const std::string &soundName) const;
 
         int layerOrder() const;

src/scratch/target.cpp

@@ -221,7 +221,7 @@ int Target::addCostume(const Costume &costume)
 }
 
 /*! Returns the costume at index. */
-Costume Target::costumeAt(int index) const
+const libscratchcpp::Costume &Target::costumeAt(int index) const
 {
     // TODO: Add range check
     return impl->costumes[index];
@@ -267,7 +267,7 @@ int Target::addSound(const Sound &sound)
 }
 
 /*! Returns the sound at index. */
-Sound Target::soundAt(int index) const
+const libscratchcpp::Sound &Target::soundAt(int index) const
 {
     // TODO: Add range check
     return impl->sounds[index];