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

Unified Diff: pkg/compiler/lib/src/dart_backend/outputter.dart

Issue 1383483006: Extract DiagnosticReporter implementation from Compiler. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 2 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: pkg/compiler/lib/src/dart_backend/outputter.dart
diff --git a/pkg/compiler/lib/src/dart_backend/outputter.dart b/pkg/compiler/lib/src/dart_backend/outputter.dart
index a8240adf14acfe59a65b23f5cefc1b2b689f2720..6b373ad9bb3359ba2086cf87d9c732813b1b4090 100644
--- a/pkg/compiler/lib/src/dart_backend/outputter.dart
+++ b/pkg/compiler/lib/src/dart_backend/outputter.dart
@@ -18,7 +18,7 @@ typedef List<Element> ElementSorter(Iterable<Element> elements);
/// Output engine for dart2dart that is shared between the dart2js and the
/// analyzer implementations of dart2dart.
class DartOutputter {
- final DiagnosticListener listener;
+ final DiagnosticReporter listener;
sigurdm 2015/10/05 13:45:34 Rename variable? (several occurences in this file)
Johnni Winther 2015/10/06 08:04:32 Done.
final CompilerOutputProvider outputProvider;
final bool forceStripTypes;
@@ -147,7 +147,7 @@ class DartOutputter {
}
static PlaceholderCollector collectPlaceholders(
- DiagnosticListener listener,
+ DiagnosticReporter listener,
MirrorRenamer mirrorRenamer,
FunctionElement mainFunction,
LibraryInfo libraryInfo,
@@ -194,7 +194,7 @@ class DartOutputter {
return placeholderRenamer;
}
- static String astOutput(DiagnosticListener listener,
+ static String astOutput(DiagnosticReporter listener,
ElementInfo elementInfo) {
// TODO(antonm): Ideally XML should be a separate backend.
// TODO(antonm): obey renames and minification, at least as an option.
@@ -229,7 +229,7 @@ class LibraryInfo {
this.userLibraries);
static LibraryInfo processLibraries(
- DiagnosticListener listener,
+ DiagnosticReporter reporter,
Iterable<LibraryElement> libraries,
Iterable<AstElement> resolvedElements) {
Set<String> fixedStaticNames = new Set<String>();
@@ -299,14 +299,14 @@ class LibraryInfo {
ClassElement existingEnumClass =
enumClassMap.putIfAbsent(cls.name, () => cls);
if (existingEnumClass != cls) {
- listener.reportError(
- listener.createMessage(
+ reporter.reportError(
+ reporter.createMessage(
cls,
MessageKind.GENERIC,
{'text': "Duplicate enum names are not supported "
"in dart2dart."}),
<DiagnosticMessage>[
- listener.createMessage(
+ reporter.createMessage(
existingEnumClass,
MessageKind.GENERIC,
{'text': "This is the other declaration of '${cls.name}'."}),

Powered by Google App Engine
This is Rietveld 408576698