Do not export library prefix names used in a library

runtime/vm/object.cc

 // Copyright (c) 2012, 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/object.h"
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/cpu.h"
@@ skipping 7675 matching lines @@
   return false;
 }
 
 
 RawObject* Namespace::Lookup(const String& name) const {
   Isolate* isolate = Isolate::Current();
   const Library& lib = Library::Handle(isolate, library());
   intptr_t ignore = 0;
   // Lookup the name in the library's symbols.
   Object& obj = Object::Handle(isolate, lib.LookupEntry(name, &ignore));
-  if (obj.IsNull()) {
+  // Library prefixes are not exported.
+  if (obj.IsNull() || obj.IsLibraryPrefix()) {
     // Lookup in the re-exported symbols.
     obj = lib.LookupExport(name);

[Ivan Posva, 2013/09/03 21:56:06]

Wouldn't it be a conflict if the library is re-exporting something which
conflicts with a library prefix?

As far as I understand this will just cause us to overwrite the found library
prefix with a null from LookupExport, but it feels a bit weird.

[hausner, 2013/09/03 22:19:10]

That is part of what Gilad has to clarify for us. If a library prefix does not
belong to the exported namespace, then there can be no conflict with a
re-exported name. In other words, library L can contain a prefix P and may
re-export a class P from another library L2.

The IsLibraryPrefix() check in line 7701 below is actually redundant. We should
never get a library prefix back from lib.LookupExport().

   }
-  if (obj.IsNull() || HidesName(name)) {
+  if (obj.IsNull() || HidesName(name) || obj.IsLibraryPrefix()) {
     return Object::null();
   }
   return obj.raw();
 }
 
 
 RawNamespace* Namespace::New() {
   ASSERT(Object::namespace_class() != Class::null());
   RawObject* raw = Object::Allocate(Namespace::kClassId,
                                     Namespace::InstanceSize(),
@@ skipping 7189 matching lines @@
 }
 
 
 void MirrorReference::PrintToJSONStream(JSONStream* stream, bool ref) const {
   stream->OpenObject();
   stream->CloseObject();
 }
 
 
 }  // namespace dart