Add unsafeConstants parameter to Compiler constructor

Author: adazem009

include/scratchcpp/compiler.h

@@ -53,7 +53,7 @@ class LIBSCRATCHCPP_EXPORT Compiler
         using Args = std::vector<CompilerValue *>;
 
         Compiler(CompilerContext *ctx);
-        Compiler(IEngine *engine, Target *target);
+        Compiler(IEngine *engine, Target *target, const std::unordered_set<std::string> &unsafeConstants = {});
         Compiler(const Compiler &) = delete;
 
         IEngine *engine() const;

src/engine/compiler.cpp

@@ -20,8 +20,8 @@ Compiler::Compiler(CompilerContext *ctx) :
 }
 
 /*! Constructs Compiler using a new context for the given target. */
-Compiler::Compiler(IEngine *engine, Target *target) :
-    impl(spimpl::make_unique_impl<CompilerPrivate>(engine, target))
+Compiler::Compiler(IEngine *engine, Target *target, const std::unordered_set<std::string> &unsafeConstants) :
+    impl(spimpl::make_unique_impl<CompilerPrivate>(engine, target, unsafeConstants))
 {
 }
 

src/engine/compiler_p.cpp

@@ -17,8 +17,8 @@ CompilerPrivate::CompilerPrivate(CompilerContext *ctx) :
     initBuilderFactory();
 }
 
-CompilerPrivate::CompilerPrivate(IEngine *engine, Target *target) :
-    ctxPtr(Compiler::createContext(engine, target, {})), // NOTE: This constructor is only used in tests, so we can leave unsafe constants empty
+CompilerPrivate::CompilerPrivate(IEngine *engine, Target *target, const std::unordered_set<std::string> &unsafeConstants) :
+    ctxPtr(Compiler::createContext(engine, target, unsafeConstants)),
     ctx(ctxPtr.get())
 {
     initBuilderFactory();

src/engine/compiler_p.h

@@ -29,7 +29,7 @@ class LIBSCRATCHCPP_TEST_EXPORT CompilerPrivate
         };
 
         CompilerPrivate(CompilerContext *ctx);
-        CompilerPrivate(IEngine *engine, Target *target);
+        CompilerPrivate(IEngine *engine, Target *target, const std::unordered_set<std::string> &unsafeConstants);
 
         static void initBuilderFactory();