Lazily format LanguageError messages (fix issue 15069).

runtime/vm/flow_graph_builder.cc

Index: runtime/vm/flow_graph_builder.cc
===================================================================
--- runtime/vm/flow_graph_builder.cc	(revision 30298)
+++ runtime/vm/flow_graph_builder.cc	(working copy)
@@ -3882,12 +3882,17 @@
 
 void FlowGraphBuilder::Bailout(const char* reason) {
   const char* kFormat = "FlowGraphBuilder Bailout: %s %s";
-  const char* function_name = parsed_function_->function().ToCString();
+  const Function& function = parsed_function_->function();
+  const char* function_name = function.ToCString();
   intptr_t len = OS::SNPrint(NULL, 0, kFormat, function_name, reason) + 1;
   char* chars = Isolate::Current()->current_zone()->Alloc<char>(len);
   OS::SNPrint(chars, len, kFormat, function_name, reason);
   const Error& error = Error::Handle(
-      LanguageError::New(String::Handle(String::New(chars))));
+      LanguageError::New(Error::Handle(),  // No previous error.
+                         Script::Handle(function.script()),
+                         function.token_pos(),
+                         LanguageError::kError,
+                         String::Handle(String::New(chars))));

[Ivan Posva, 2013/11/15 04:42:13]

ditto.

[regis, 2013/11/15 18:27:38]

Done.

   Isolate::Current()->long_jump_base()->Jump(1, error);
 }