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

Unified Diff: pkg/kernel/lib/analyzer/loader.dart

Issue 2614663007: Revert "Non-format-changing kernel offset changes" (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « pkg/kernel/lib/analyzer/ast_from_analyzer.dart ('k') | pkg/kernel/lib/ast.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/kernel/lib/analyzer/loader.dart
diff --git a/pkg/kernel/lib/analyzer/loader.dart b/pkg/kernel/lib/analyzer/loader.dart
index cba5824929c67f11962020127e6b9fcf8f7fe09a..43a1b3dd41f31055c6a0e81bafb19d4ae56108b9 100644
--- a/pkg/kernel/lib/analyzer/loader.dart
+++ b/pkg/kernel/lib/analyzer/loader.dart
@@ -268,7 +268,7 @@ class DartLoader implements ReferenceLevelLoader {
_classes[element] = classNode = new ast.Class(
name: element.name,
isAbstract: element.isAbstract,
- fileUri: '${element.source.uri}')..fileOffset = element.nameOffset;
+ fileUri: '${element.source.uri}');
classNode.level = ast.ClassLevel.Temporary;
var library = getLibraryReference(element.library);
library.addClass(classNode);
@@ -318,9 +318,6 @@ class DartLoader implements ReferenceLevelLoader {
// Use a shared mixin application class for this library.
var mixinClass = getSharedMixinApplicationClass(
scope.currentLibrary, supertype.classNode, mixinType.classNode);
- if (mixinClass.fileOffset < 0) {
- mixinClass.fileOffset = element.nameOffset;
- }
supertype = new ast.Supertype(
mixinClass,
supertype.typeArguments.length > mixinType.typeArguments.length
@@ -336,7 +333,7 @@ class DartLoader implements ReferenceLevelLoader {
typeParameters: freshParameters.freshTypeParameters,
supertype: freshParameters.substituteSuper(supertype),
mixedInType: freshParameters.substituteSuper(mixinType),
- fileUri: classNode.fileUri)..fileOffset = element.nameOffset;
+ fileUri: classNode.fileUri);
mixinClass.level = ast.ClassLevel.Type;
supertype = new ast.Supertype(mixinClass,
classNode.typeParameters.map(makeTypeParameterType).toList());
@@ -449,14 +446,12 @@ class DartLoader implements ReferenceLevelLoader {
isStatic: true,
isExternal: constructor.isExternal,
isConst: constructor.isConst,
- fileUri: '${element.source.uri}')
- ..fileOffset = element.nameOffset;
+ fileUri: '${element.source.uri}');
}
return new ast.Constructor(scope.buildFunctionInterface(constructor),
name: _nameOfMember(element),
isConst: constructor.isConst,
- isExternal: constructor.isExternal)
- ..fileOffset = element.nameOffset;
+ isExternal: constructor.isExternal);
case ElementKind.FIELD:
case ElementKind.TOP_LEVEL_VARIABLE:
@@ -485,7 +480,7 @@ class DartLoader implements ReferenceLevelLoader {
isAbstract: executable.isAbstract,
isStatic: executable.isStatic,
isExternal: executable.isExternal,
- fileUri: '${element.source.uri}')..fileOffset = element.nameOffset;
+ fileUri: '${element.source.uri}');
default:
throw 'Unexpected member kind: $element';
« no previous file with comments | « pkg/kernel/lib/analyzer/ast_from_analyzer.dart ('k') | pkg/kernel/lib/ast.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698