Remove Compiler::breakAtomicScript()

Author: adazem009

include/scratchcpp/compiler.h

@@ -62,7 +62,6 @@ class LIBSCRATCHCPP_EXPORT Compiler
         void addProcedureArg(const std::string &procCode, const std::string &argName);
         void moveToSubstack(std::shared_ptr<Block> substack1, std::shared_ptr<Block> substack2, SubstackType type);
         void moveToSubstack(std::shared_ptr<Block> substack, SubstackType type);
-        void breakAtomicScript();
         void warp();
 
         Input *input(int id) const;

src/engine/compiler.cpp

@@ -27,7 +27,6 @@ void Compiler::init()
 
     impl->bytecode.clear();
     impl->procedurePrototype = nullptr;
-    impl->atomic = true;
     impl->warp = false;
 
     // Add start instruction
@@ -235,16 +234,6 @@ void Compiler::moveToSubstack(std::shared_ptr<Block> substack, SubstackType type
     moveToSubstack(substack, nullptr, type);
 }
 
-/*!
- * Adds the vm::OP_BREAK_ATOMIC instruction at the end of the current loop.
- * This can be used for example in motion blocks.
- * \note Nothing will happen if the script is set to run without screen refresh.
- */
-void Compiler::breakAtomicScript()
-{
-    impl->atomic = false;
-}
-
 /*! Makes current script run without screen refresh. */
 void Compiler::warp()
 {

src/engine/compiler_p.h

@@ -36,7 +36,6 @@ struct CompilerPrivate
         std::vector<std::string> procedures;
         std::unordered_map<std::string, std::vector<std::string>> procedureArgs;
         BlockPrototype *procedurePrototype = nullptr;
-        bool atomic = true;
         bool warp = false;
 };
 

test/compiler/compiler_test.cpp

@@ -430,21 +430,6 @@ TEST_F(CompilerTest, AddFunctionCall)
     ASSERT_EQ(engine.functionIndex(&testFunction2), 1);
 }
 
-TEST_F(CompilerTest, BreakAtomicScript)
-{
-    static const int SUBSTACK = 0;
-
-    INIT_COMPILER(engine, compiler);
-    compiler.addInstruction(vm::OP_START);
-
-    compiler.addInstruction(vm::OP_FOREVER_LOOP);
-    compiler.breakAtomicScript();
-    compiler.moveToSubstack(nullptr, Compiler::SubstackType::Loop);
-
-    compiler.addInstruction(vm::OP_HALT);
-    ASSERT_EQ(compiler.bytecode(), std::vector<unsigned int>({ vm::OP_START, vm::OP_FOREVER_LOOP, vm::OP_BREAK_ATOMIC, vm::OP_LOOP_END, vm::OP_HALT }));
-}
-
 TEST_F(CompilerTest, Warp)
 {
     INIT_COMPILER(engine, compiler);