Remove broadcast running methods from IEngine

Author: adazem009

include/scratchcpp/iengine.h

@@ -207,12 +207,6 @@ class LIBSCRATCHCPP_EXPORT IEngine
         /*! Toggles sprite fencing. */
         virtual void setSpriteFencingEnabled(bool enable) = 0;
 
-        /*! Returns true if there are any running script of the broadcast with the given index. */
-        virtual bool broadcastRunning(unsigned int index) = 0;
-
-        /*! Returns true if there are any running script of the given broadcast. */
-        virtual bool broadcastByPtrRunning(Broadcast *broadcast) = 0;
-
         /*!
          * Call this from a block implementation to force a redraw (screen refresh).
          * \note This has no effect in "run without screen refresh" custom blocks.

src/engine/internal/engine.cpp

@@ -960,57 +960,6 @@ void Engine::setSpriteFencingEnabled(bool enable)
     m_spriteFencingEnabled = enable;
 }
 
-bool Engine::broadcastRunning(unsigned int index)
-{
-    if (index < 0 || index >= m_broadcasts.size())
-        return false;
-
-    return broadcastByPtrRunning(m_broadcasts[index].get());
-}
-
-bool Engine::broadcastByPtrRunning(Broadcast *broadcast)
-{
-    if (broadcast->isBackdropBroadcast()) {
-        // This broadcast belongs to a backdrop
-        assert(m_broadcastMap.find(broadcast) == m_broadcastMap.cend());
-
-        for (auto thread : m_threads) {
-            if (!thread->atEnd()) {
-                Script *script = thread->script();
-                auto topBlock = script->topBlock();
-
-                const auto &scripts = m_backdropChangeHats[script->target()];
-                auto scriptIt = std::find(scripts.begin(), scripts.end(), script);
-                auto scriptFieldMapIt = m_scriptHatFields.find(script);
-
-                if (scriptFieldMapIt != m_scriptHatFields.cend()) {
-                    const auto &fieldMap = scriptFieldMapIt->second;
-                    auto fieldIt = fieldMap.find(HatField::Backdrop);
-                    assert(fieldIt != fieldMap.cend());
-                    assert(topBlock->findFieldById(fieldIt->second));
-
-                    if ((scriptIt != scripts.end()) && (topBlock->findFieldById(fieldIt->second)->value().toString() == broadcast->name()))
-                        return true;
-                }
-            }
-        }
-    } else {
-        // This is a regular broadcast
-        assert(m_broadcastMap.find(broadcast) != m_broadcastMap.cend());
-        const auto &scripts = m_broadcastMap[broadcast];
-
-        for (auto thread : m_threads) {
-            if (!thread->atEnd()) {
-                auto it = std::find_if(scripts.begin(), scripts.end(), [thread](Script *script) { return thread->script() == script; });
-
-                if (it != scripts.end())
-                    return true;
-            }
-        }
-    }
-    return false;
-}
-
 void Engine::requestRedraw()
 {
     m_redrawRequested = true;

src/engine/internal/engine.h

@@ -98,9 +98,6 @@ class Engine : public IEngine
         bool spriteFencingEnabled() const override;
         void setSpriteFencingEnabled(bool enable) override;
 
-        bool broadcastRunning(unsigned int index) override;
-        bool broadcastByPtrRunning(Broadcast *broadcast) override;
-
         void requestRedraw() override;
 
         ITimer *timer() const override;

test/mocks/enginemock.h

@@ -80,9 +80,6 @@ class EngineMock : public IEngine
         MOCK_METHOD(bool, spriteFencingEnabled, (), (const, override));
         MOCK_METHOD(void, setSpriteFencingEnabled, (bool), (override));
 
-        MOCK_METHOD(bool, broadcastRunning, (unsigned int), (override));
-        MOCK_METHOD(bool, broadcastByPtrRunning, (Broadcast *), (override));
-
         MOCK_METHOD(void, requestRedraw, (), (override));
 
         MOCK_METHOD(ITimer *, timer, (), (const, override));