Make compile-time errors catchable

runtime/vm/exceptions.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/exceptions.h"
 
 #include "platform/address_sanitizer.h"
 
 #include "vm/dart_api_impl.h"
 #include "vm/dart_entry.h"
@@ skipping 622 matching lines @@
                                  intptr_t expected_to) {
   const Array& args = Array::Handle(Array::New(4));
   args.SetAt(0, argument_value);
   args.SetAt(1, Integer::Handle(Integer::New(expected_from)));
   args.SetAt(2, Integer::Handle(Integer::New(expected_to)));
   args.SetAt(3, String::Handle(String::New(argument_name)));
   Exceptions::ThrowByType(Exceptions::kRange, args);
 }
 
 
+void Exceptions::ThrowSyntaxError(const LanguageError& error) {
+  const Array& args = Array::Handle(Array::New(1));
+  args.SetAt(0, String::Handle(error.FormatMessage()));
+  Exceptions::ThrowByType(Exceptions::kSyntaxError, args);
+}
+
+
 RawObject* Exceptions::Create(ExceptionType type, const Array& arguments) {
   Library& library = Library::Handle();
   const String* class_name = NULL;
   const String* constructor_name = &Symbols::Dot();
   switch (type) {
     case kNone:
     case kStackOverflow:
     case kOutOfMemory:
       UNREACHABLE();
       break;
@@ skipping 53 matching lines @@
       constructor_name = &Symbols::DotCreate();
       break;
     case kAbstractClassInstantiation:
       library = Library::CoreLibrary();
       class_name = &Symbols::AbstractClassInstantiationError();
       constructor_name = &Symbols::DotCreate();
       break;
     case kCyclicInitializationError:
       library = Library::CoreLibrary();
       class_name = &Symbols::CyclicInitializationError();
+      break;
+    case kSyntaxError:
+      library = Library::CoreLibrary();
+      class_name = &Symbols::SyntaxError();

[siva, 2016/09/15 18:11:45]

Does this need to be
  break;

default:
  UNREACHABLE();

like the way we do in other switch statements?

[hausner, 2016/09/15 21:28:28]

Done.

There does not seem to be a default case, though.

   }
 
   return DartLibraryCalls::InstanceCreate(library,
                                           *class_name,
                                           *constructor_name,
                                           arguments);
 }
 
 
 }  // namespace dart