Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Unified Diff: sdk/lib/_internal/compiler/implementation/library_loader.dart

Issue 266913017: Convert property methods into getters. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Rebased Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sdk/lib/_internal/compiler/implementation/library_loader.dart
diff --git a/sdk/lib/_internal/compiler/implementation/library_loader.dart b/sdk/lib/_internal/compiler/implementation/library_loader.dart
index 6592d2a1fc1b3c36228f0b5e3839aefb99498d44..7ebe1f312b791461c09e18e848b1f8d0dc2d31df 100644
--- a/sdk/lib/_internal/compiler/implementation/library_loader.dart
+++ b/sdk/lib/_internal/compiler/implementation/library_loader.dart
@@ -551,7 +551,7 @@ class ImportLink {
String prefix = import.prefix.source;
Element existingElement = importingLibrary.find(prefix);
PrefixElement prefixElement;
- if (existingElement == null || !existingElement.isPrefix()) {
+ if (existingElement == null || !existingElement.isPrefix) {
prefixElement = new PrefixElementX(prefix,
importingLibrary.entryCompilationUnit, import.getBeginToken());
} else {
@@ -739,7 +739,7 @@ class LibraryDependencyNode {
Link<Export> duplicateExports,
{bool reportError: true}) {
assert(invariant(library, !duplicateExports.isEmpty,
- message: "No export for $duplicate from ${duplicate.getLibrary()} "
+ message: "No export for $duplicate from ${duplicate.library} "
"in $library."));
compiler.withCurrentElement(library, () {
for (Export export in duplicateExports) {
@@ -758,7 +758,7 @@ class LibraryDependencyNode {
void reportDuplicateExportDecl(Element duplicate,
Link<Export> duplicateExports) {
assert(invariant(library, !duplicateExports.isEmpty,
- message: "No export for $duplicate from ${duplicate.getLibrary()} "
+ message: "No export for $duplicate from ${duplicate.library} "
"in $library."));
compiler.reportInfo(duplicate, MessageKind.DUPLICATE_EXPORT_DECL,
{'name': name, 'uriString': duplicateExports.head.uri});
@@ -766,13 +766,13 @@ class LibraryDependencyNode {
Element existingElement = exportScope[name];
if (existingElement != null && existingElement != element) {
- if (existingElement.isErroneous()) {
+ if (existingElement.isErroneous) {
reportDuplicateExport(element, exports);
reportDuplicateExportDecl(element, exports);
element = existingElement;
- } else if (existingElement.getLibrary() == library) {
+ } else if (existingElement.library == library) {
// Do nothing. [existingElement] hides [element].
- } else if (element.getLibrary() == library) {
+ } else if (element.library == library) {
// [element] hides [existingElement].
exportScope[name] = element;
exporters[element] = exports;

Powered by Google App Engine
This is Rietveld 408576698