Store edge-activated hat values by block and target

Author: adazem009

src/engine/internal/engine.cpp

@@ -506,39 +506,23 @@ void Engine::step()
                 auto hatBlock = thread->script()->topBlock();
                 assert(hatBlock);
 
-                // TODO: Edge-activated hats currently support only one field
-                // If there isn't any field, -1 is used as the field value ID
-                int fieldValueId = -1;
-
-                if (hatBlock->fieldAt(0)) {
-                    fieldValueId = hatBlock->fieldAt(0)->specialValueId();
-                    assert(fieldValueId != -1);
-                }
-
                 Target *target = hatBlock->target();
                 assert(target);
-                auto it = m_edgeActivatedHatValues.find(hatType);
+                auto it = m_edgeActivatedHatValues.find(hatBlock.get());
 
                 if (it == m_edgeActivatedHatValues.cend()) {
-                    m_edgeActivatedHatValues[hatType] = { { target, {} } };
+                    m_edgeActivatedHatValues[hatBlock.get()] = {};
                 } else {
                     auto &map = it->second;
                     auto it = map.find(target);
 
-                    if (it == map.cend())
-                        map[target] = {};
-                    else {
-                        const std::unordered_map<int, bool> &values = it->second;
-                        auto fieldIt = values.find(fieldValueId);
-
-                        if (fieldIt != values.cend())
-                            oldValue = fieldIt->second;
-                    }
+                    if (it != map.cend())
+                        oldValue = it->second;
                 }
 
                 bool newValue = thread->script()->runHatPredicate();
                 bool edgeWasActivated = !oldValue && newValue; // changed from false true
-                m_edgeActivatedHatValues[hatType][target][fieldValueId] = newValue;
+                m_edgeActivatedHatValues[hatBlock.get()][target] = newValue;
 
                 if (!edgeWasActivated)
                     stopThread(thread.get());

src/engine/internal/engine.h

@@ -246,8 +246,7 @@ class Engine : public IEngine
 
         std::unordered_map<Script *, std::unordered_map<HatField, int>> m_scriptHatFields; // HatField, field ID from the block implementation
 
-        std::unordered_map<HatType, std::unordered_map<Target *, std::unordered_map<int, bool>>> m_edgeActivatedHatValues; // (target, field value ID, last value) edge-activated hats only run after
-                                                                                                                           // the value changes from false to true
+        std::unordered_map<Block *, std::unordered_map<Target *, bool>> m_edgeActivatedHatValues; // (block, target, last value) edge-activated hats only run after the value changes from false to true
 
         std::unique_ptr<ITimer> m_defaultTimer;
         ITimer *m_timer = nullptr;