fix: use dynamically growing error buffer in ValidateScript

GodGun123 · GodGun123 · commit 4df833b5a0ad · 2025-09-10T09:40:41.000Z
diff --git a/src/script_parser.cpp b/src/script_parser.cpp
@@ -70,11 +70,12 @@ BT::Expected<Any> ParseScriptAndExecute(Ast::Environment& env, const std::string
 
 Result ValidateScript(const std::string& script)
 {
-  char error_msgs_buffer[2048];
+  std::string error_msgs_buffer;  // dynamically growing error buffer
 
   auto input = lexy::string_input<lexy::utf8_encoding>(script);
-  auto result =
-      lexy::parse<BT::Grammar::stmt>(input, ErrorReport().to(error_msgs_buffer));
+
+  auto reporter = ErrorReport().to(std::back_inserter(error_msgs_buffer));
+  auto result = lexy::parse<BT::Grammar::stmt>(input, reporter);
   if(result.has_value() && result.error_count() == 0)
   {
     try

Original file line number	Diff line number	Diff line change
`@@ -70,11 +70,12 @@ BT::Expected<Any> ParseScriptAndExecute(Ast::Environment& env, const std::string`
`70`	`70`
`71`	`71`	`Result ValidateScript(const std::string& script)`
`72`	`72`	`{`
`73`		`- char error_msgs_buffer[2048];`
	`73`	`+ std::string error_msgs_buffer; // dynamically growing error buffer`
`74`	`74`
`75`	`75`	`auto input = lexy::string_input<lexy::utf8_encoding>(script);`
`76`		`- auto result =`
`77`		`- lexy::parse<BT::Grammar::stmt>(input, ErrorReport().to(error_msgs_buffer));`
	`76`	`+`
	`77`	`+ auto reporter = ErrorReport().to(std::back_inserter(error_msgs_buffer));`
	`78`	`+ auto result = lexy::parse<BT::Grammar::stmt>(input, reporter);`
`78`	`79`	`if(result.has_value() && result.error_count() == 0)`
`79`	`80`	`{`
`80`	`81`	`try`