adding Ark::State::doString, thus updating REPL to use the new State API

Author: SuperFola

include/Ark/VM/State.hpp

@@ -25,6 +25,8 @@ namespace Ark
 
         // to compile file *only* if needed and use the resulting bytecode
         bool doFile(const std::string& filename);
+        // compile string and store resulting bytecode in m_bytecode
+        bool doString(const std::string& code);
 
         void loadFunction(const std::string& name, internal::Value::ProcType function);
         void setDebug(bool value);

src/REPL/Repl.cpp

@@ -9,9 +9,8 @@ namespace Ark
 
         print_repl_header();
 
-        while(true)
+        while (true)
         {
-            Ark::Compiler compiler(false, m_lib_dir, m_options);
             Ark::State state(m_lib_dir, /* filename */ "FILE", m_options);
             Ark::VM vm(&state, m_options);
             state.setDebug(false);
@@ -48,25 +47,10 @@ namespace Ark
                 std::cout << continuing_prompt;
             }
 
-            try
-            {
-                compiler.feed("{" + code.str() + "}");
-                compiler.compile();
-
-                state.feed(compiler.bytecode());
+            if (state.doString("{" + code.str() + "}"))
                 vm.run();
-            }
-            catch (const std::runtime_error& e) {
-                std::cerr << e.what() << std::endl;
-            }
-            catch (const std::exception& e)
-            {
-                std::cerr << e.what() << std::endl;
-            }
-            catch (...)
-            {
-                std::cerr << "Unknown lexer-parser-or-compiler error" << std::endl;
-            }
+            else
+                std::cerr << "Ark::State::doString failed" << std::endl;
         }
     }
 

src/VM/State.cpp

@@ -142,6 +142,31 @@ namespace Ark
         return false;
     }
 
+    bool State::doString(const std::string& code)
+    {
+        Compiler compiler(m_debug, m_libdir, m_options);
+
+        try
+        {
+            compiler.feed(code);
+            compiler.compile();
+        }
+        catch (const std::exception& e)
+        {
+            std::cerr << e.what() << std::endl;
+            return false;
+        }
+        catch (...)
+        {
+            std::cerr << "Unknown lexer-parser-or-compiler error" << std::endl;
+            return false;
+        }
+
+        state.feed(compiler.bytecode());
+
+        return true;
+    }
+
     void State::loadFunction(const std::string& name, internal::Value::ProcType function)
     {
         m_binded_functions[name] = std::move(function);