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

Unified Diff: pkg/analyzer/lib/src/task/dart.dart

Issue 1131423002: Clean up many generated constructors (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Comment change Created 5 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: pkg/analyzer/lib/src/task/dart.dart
diff --git a/pkg/analyzer/lib/src/task/dart.dart b/pkg/analyzer/lib/src/task/dart.dart
index 3e1240992993b5af6f5cf7d2c488acd4e2812daf..00686d6d0e077dabfc9c958b68138edf3ad8ae62 100644
--- a/pkg/analyzer/lib/src/task/dart.dart
+++ b/pkg/analyzer/lib/src/task/dart.dart
@@ -427,7 +427,7 @@ class BuildClassConstructorsTask extends SourceBasedAnalysisTask {
}
if (_findForwardedConstructors(classElement, superType, callback)) {
if (implicitConstructors.isEmpty) {
- errors.add(new AnalysisError.con2(classElement.source,
+ errors.add(new AnalysisError(classElement.source,
classElement.nameOffset, classElement.name.length,
CompileTimeErrorCode.MIXIN_HAS_NO_CONSTRUCTORS,
[superType.element.name]));
@@ -450,7 +450,7 @@ class BuildClassConstructorsTask extends SourceBasedAnalysisTask {
if (_findForwardedConstructors(classElement, superType, callback) &&
!constructorFound) {
SourceRange withRange = classElement.withClauseRange;
- errors.add(new AnalysisError.con2(classElement.source, withRange.offset,
+ errors.add(new AnalysisError(classElement.source, withRange.offset,
withRange.length, CompileTimeErrorCode.MIXIN_HAS_NO_CONSTRUCTORS,
[superType.element.name]));
classElement.mixinErrorsReported = true;
@@ -529,7 +529,7 @@ class BuildClassConstructorsTask extends SourceBasedAnalysisTask {
}
implicitConstructor.parameters = implicitParameters;
}
- FunctionTypeImpl type = new FunctionTypeImpl.con1(implicitConstructor);
+ FunctionTypeImpl type = new FunctionTypeImpl(implicitConstructor);
type.typeArguments = classType.typeArguments;
implicitConstructor.type = type;
return implicitConstructor;
@@ -609,10 +609,7 @@ class BuildCompilationUnitElementTask extends SourceBasedAnalysisTask {
*/
static final TaskDescriptor DESCRIPTOR = new TaskDescriptor(
'BuildCompilationUnitElementTask', createTask, buildInputs,
- <ResultDescriptor>[
- COMPILATION_UNIT_ELEMENT,
- RESOLVED_UNIT1
- ]);
+ <ResultDescriptor>[COMPILATION_UNIT_ELEMENT, RESOLVED_UNIT1]);
/**
* Initialize a newly created task to build a compilation unit element for
@@ -785,9 +782,8 @@ class BuildDirectiveElementsTask extends SourceBasedAnalysisTask {
ErrorCode errorCode = (importElement.isDeferred
? StaticWarningCode.IMPORT_OF_NON_LIBRARY
: CompileTimeErrorCode.IMPORT_OF_NON_LIBRARY);
- errors.add(new AnalysisError.con2(importedSource,
- uriLiteral.offset, uriLiteral.length, errorCode,
- [uriLiteral.toSource()]));
+ errors.add(new AnalysisError(importedSource, uriLiteral.offset,
+ uriLiteral.length, errorCode, [uriLiteral.toSource()]));
}
}
}
@@ -812,9 +808,8 @@ class BuildDirectiveElementsTask extends SourceBasedAnalysisTask {
directive.element = exportElement;
exports.add(exportElement);
if (exportSourceKindMap[exportedSource] != SourceKind.LIBRARY) {
- errors.add(new AnalysisError.con2(exportedSource,
- uriLiteral.offset, uriLiteral.length,
- CompileTimeErrorCode.EXPORT_OF_NON_LIBRARY,
+ errors.add(new AnalysisError(exportedSource, uriLiteral.offset,
+ uriLiteral.length, CompileTimeErrorCode.EXPORT_OF_NON_LIBRARY,
[uriLiteral.toSource()]));
}
}
@@ -1276,7 +1271,7 @@ class BuildLibraryElementTask extends SourceBasedAnalysisTask {
String partLibraryName =
_getPartLibraryName(partSource, partUnit, directivesToResolve);
if (partLibraryName == null) {
- errors.add(new AnalysisError.con2(librarySource, partUri.offset,
+ errors.add(new AnalysisError(librarySource, partUri.offset,
partUri.length, CompileTimeErrorCode.PART_OF_NON_PART,
[partUri.toSource()]));
} else if (libraryNameNode == null) {
@@ -1286,7 +1281,7 @@ class BuildLibraryElementTask extends SourceBasedAnalysisTask {
partsLibraryName = null;
}
} else if (libraryNameNode.name != partLibraryName) {
- errors.add(new AnalysisError.con2(librarySource, partUri.offset,
+ errors.add(new AnalysisError(librarySource, partUri.offset,
partUri.length, StaticWarningCode.PART_OF_DIFFERENT_LIBRARY, [
libraryNameNode.name,
partLibraryName
@@ -1304,11 +1299,11 @@ class BuildLibraryElementTask extends SourceBasedAnalysisTask {
AnalysisError error;
if (partsLibraryName != _UNKNOWN_LIBRARY_NAME &&
partsLibraryName != null) {
- error = new AnalysisErrorWithProperties.con1(librarySource,
+ error = new AnalysisErrorWithProperties(librarySource, 0, 0,
ResolverErrorCode.MISSING_LIBRARY_DIRECTIVE_WITH_PART)
..setProperty(ErrorProperty.PARTS_LIBRARY_NAME, partsLibraryName);
} else {
- error = new AnalysisError.con1(librarySource,
+ error = new AnalysisError(librarySource, 0, 0,
ResolverErrorCode.MISSING_LIBRARY_DIRECTIVE_WITH_PART);
}
errors.add(error);
@@ -2509,15 +2504,13 @@ class ParseDartTask extends SourceBasedAnalysisTask {
return null;
}
if (code == UriValidationCode.URI_WITH_INTERPOLATION) {
- errorListener.onError(new AnalysisError.con2(librarySource,
- uriLiteral.offset, uriLiteral.length,
- CompileTimeErrorCode.URI_WITH_INTERPOLATION));
+ errorListener.onError(new AnalysisError(librarySource, uriLiteral.offset,
+ uriLiteral.length, CompileTimeErrorCode.URI_WITH_INTERPOLATION));
return null;
}
if (code == UriValidationCode.INVALID_URI) {
- errorListener.onError(new AnalysisError.con2(librarySource,
- uriLiteral.offset, uriLiteral.length,
- CompileTimeErrorCode.INVALID_URI, [uriContent]));
+ errorListener.onError(new AnalysisError(librarySource, uriLiteral.offset,
+ uriLiteral.length, CompileTimeErrorCode.INVALID_URI, [uriContent]));
return null;
}
throw new AnalysisException('Failed to handle validation code: $code');
« no previous file with comments | « pkg/analyzer/lib/src/generated/testing/element_factory.dart ('k') | pkg/analyzer/test/generated/all_the_rest_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698