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

Unified Diff: pkg/front_end/lib/src/fasta/source/outline_builder.dart

Issue 2970273004: Deprecate all diagnostics methods that use strings. (Closed)
Patch Set: Created 3 years, 5 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/front_end/lib/src/fasta/source/outline_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/source/outline_builder.dart b/pkg/front_end/lib/src/fasta/source/outline_builder.dart
index 5f01274ac372eba64136889fb58f675d4620bf24..65f79ed30ee154abebce8b459600a4cff5a828fa 100644
--- a/pkg/front_end/lib/src/fasta/source/outline_builder.dart
+++ b/pkg/front_end/lib/src/fasta/source/outline_builder.dart
@@ -18,7 +18,7 @@ import '../util/link.dart' show Link;
import '../combinator.dart' show Combinator;
-import '../errors.dart' show internalError;
+import '../deprecated_problems.dart' show deprecated_internalProblem;
import '../builder/builder.dart';
@@ -201,7 +201,7 @@ class OutlineBuilder extends UnhandledListener {
push(unescapeString(token.lexeme));
push(token.charOffset);
} else {
- internalError("String interpolation not implemented.");
+ deprecated_internalProblem("String interpolation not implemented.");
}
}
@@ -290,7 +290,7 @@ class OutlineBuilder extends UnhandledListener {
if (token == null) return ProcedureKind.Method;
if (optional("get", token)) return ProcedureKind.Getter;
if (optional("set", token)) return ProcedureKind.Setter;
- return internalError("Unhandled: ${token.lexeme}");
+ return deprecated_internalProblem("Unhandled: ${token.lexeme}");
}
@override
@@ -368,7 +368,7 @@ class OutlineBuilder extends UnhandledListener {
kind = ProcedureKind.Operator;
int requiredArgumentCount = operatorRequiredArgumentCount(nameOrOperator);
if ((formals?.length ?? 0) != requiredArgumentCount) {
- library.addCompileTimeError(
+ library.deprecated_addCompileTimeError(
charOffset,
"Operator '$name' must have exactly $requiredArgumentCount "
"parameters.");
@@ -376,7 +376,7 @@ class OutlineBuilder extends UnhandledListener {
if (formals != null) {
for (FormalParameterBuilder formal in formals) {
if (!formal.isRequired) {
- library.addCompileTimeError(formal.charOffset,
+ library.deprecated_addCompileTimeError(formal.charOffset,
"An operator can't have optional parameters.");
}
}
@@ -547,9 +547,9 @@ class OutlineBuilder extends UnhandledListener {
if (formals.length == 2) {
// The name may be null for generalized function types.
if (formals[0].name != null && formals[0].name == formals[1].name) {
- library.addCompileTimeError(formals[1].charOffset,
+ library.deprecated_addCompileTimeError(formals[1].charOffset,
"Duplicated parameter name '${formals[1].name}'.");
- library.addCompileTimeError(formals[0].charOffset,
+ library.deprecated_addCompileTimeError(formals[0].charOffset,
"Other parameter named '${formals[1].name}'.");
}
} else if (formals.length > 2) {
@@ -558,9 +558,10 @@ class OutlineBuilder extends UnhandledListener {
for (FormalParameterBuilder formal in formals) {
if (formal.name == null) continue;
if (seenNames.containsKey(formal.name)) {
- library.addCompileTimeError(formal.charOffset,
+ library.deprecated_addCompileTimeError(formal.charOffset,
"Duplicated parameter name '${formal.name}'.");
- library.addCompileTimeError(seenNames[formal.name].charOffset,
+ library.deprecated_addCompileTimeError(
+ seenNames[formal.name].charOffset,
"Other parameter named '${formal.name}'.");
} else {
seenNames[formal.name] = formal;
@@ -663,7 +664,7 @@ class OutlineBuilder extends UnhandledListener {
functionType = type;
} else {
// TODO(ahe): Improve this error message.
- library.addCompileTimeError(
+ library.deprecated_addCompileTimeError(
equals.charOffset, "Can't create typedef from non-function type.");
}
}
@@ -861,7 +862,7 @@ class OutlineBuilder extends UnhandledListener {
@override
void addCompileTimeErrorFromMessage(FastaMessage message) {
- library.addCompileTimeError(message.charOffset, message.message,
+ library.deprecated_addCompileTimeError(message.charOffset, message.message,
fileUri: message.uri);
}

Powered by Google App Engine
This is Rietveld 408576698