Implement automatic loading of dart:core_native_fields library

bin/main.cc

Index: bin/main.cc
===================================================================
--- bin/main.cc	(revision 1465)
+++ bin/main.cc	(working copy)
@@ -170,7 +170,6 @@
     Dart_ExitScope();
     exit(255);
   }
-
   if (!Dart_IsLibrary(library)) {
     fprintf(stderr,
             "Expected a library when loading script: %s",
@@ -179,8 +178,6 @@
     exit(255);
   }
   Builtin_ImportLibrary(library);
-  // Setup the native resolver for built in library functions.
-  Builtin_SetNativeResolver();

[Anton Muhin, 2011/11/13 16:19:32]

why it is removed?

[siva, 2011/11/15 02:16:52]

It is not removed, it has been moved into Builtin_ImportLibrary in order to
avoid the multiple lookups.


On 2011/11/13 16:19:32, antonmuhin wrote:

 
   Dart_ExitScope();
   return data;
@@ -248,6 +245,10 @@
   }
 
   Dart_EnterScope();
+  if (snapshot_buffer != NULL) {
+    // Setup the native resolver as the snapshot does not carry it.
+    Builtin_SetNativeResolver();

[Anton Muhin, 2011/11/13 16:19:32]

why not set builtin native resolver unconditionally?  It'll be needed anyway.

[siva, 2011/11/15 02:16:52]

We don't have to do this for the regular binary as it has already been done in
Builtin_importLibrary....

On 2011/11/13 16:19:32, antonmuhin wrote:

+  }
   if (HasCompileAll(vm_options)) {
     Dart_Handle result = Dart_CompileAll();
     if (!Dart_IsValid(result)) {