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

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

Issue 22301009: Retain fewer names for reflection. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Make sure that receiver-less symbols are retained when there is insufficient @MirrorsUsed. Created 7 years, 4 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: dart/sdk/lib/_internal/compiler/implementation/mirrors_used.dart
diff --git a/dart/sdk/lib/_internal/compiler/implementation/mirrors_used.dart b/dart/sdk/lib/_internal/compiler/implementation/mirrors_used.dart
index e868ce0841e5e3cd885c30b2948ae10e672bf8e3..9c8df9cab5fbe0355c21246f47bedc7b714f2344 100644
--- a/dart/sdk/lib/_internal/compiler/implementation/mirrors_used.dart
+++ b/dart/sdk/lib/_internal/compiler/implementation/mirrors_used.dart
@@ -189,7 +189,7 @@ class MirrorUsageAnalyzer {
if (librariesWithoutUsage.isEmpty) {
karlklose 2013/08/07 15:31:20 Could we warn if there is an import with and one w
ahe 2013/08/07 15:44:17 Yes. Filed http://dartbug.com/12297.
mergedMirrorUsage = mergeUsages(usageMap);
} else {
- mergedMirrorUsage = new MirrorUsage(null, wildcard, null, null);
+ mergedMirrorUsage = new MirrorUsage(null, null, null, null);
}
}

Powered by Google App Engine
This is Rietveld 408576698