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

Side by Side Diff: pkg/analyzer/lib/src/task/incremental_element_builder.dart

Issue 2199903002: Create synthetic FieldElement(s) (don't use non-synthetic ones) for non-synthetic class getters/set… (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library analyzer.src.task.incremental_element_builder; 5 library analyzer.src.task.incremental_element_builder;
6 6
7 import 'dart:collection'; 7 import 'dart:collection';
8 8
9 import 'package:analyzer/dart/ast/ast.dart'; 9 import 'package:analyzer/dart/ast/ast.dart';
10 import 'package:analyzer/dart/ast/token.dart'; 10 import 'package:analyzer/dart/ast/token.dart';
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 oldElement.nameOffset = newElement.nameOffset; 207 oldElement.nameOffset = newElement.nameOffset;
208 oldElement.setCodeRange(newElement.codeOffset, newElement.codeLength); 208 oldElement.setCodeRange(newElement.codeOffset, newElement.codeLength);
209 oldElement.typeParameters = newElement.typeParameters; 209 oldElement.typeParameters = newElement.typeParameters;
210 } 210 }
211 // Prepare delta. 211 // Prepare delta.
212 ClassElementImpl classElement = oldClass.element; 212 ClassElementImpl classElement = oldClass.element;
213 ElementHolder classElementHolder = new ElementHolder(); 213 ElementHolder classElementHolder = new ElementHolder();
214 ClassElementDelta classDelta = 214 ClassElementDelta classDelta =
215 new ClassElementDelta(classElement, librarySource, classElement.name); 215 new ClassElementDelta(classElement, librarySource, classElement.name);
216 // Prepare all old member elements. 216 // Prepare all old member elements.
217 var removedAccessors = new Set<PropertyAccessorElement>(); 217 var removedAccessors = new Set<PropertyAccessorElement>.identity();
218 var removedConstructors = new Set<ConstructorElement>(); 218 var removedConstructors = new Set<ConstructorElement>.identity();
219 var removedMethods = new Set<MethodElement>(); 219 var removedMethods = new Set<MethodElement>.identity();
220 removedAccessors.addAll(classElement.accessors); 220 removedAccessors.addAll(classElement.accessors);
221 removedConstructors.addAll(classElement.constructors); 221 removedConstructors.addAll(classElement.constructors);
222 removedMethods.addAll(classElement.methods); 222 removedMethods.addAll(classElement.methods);
223 // Utilities. 223 // Utilities.
224 void processConstructorDeclaration( 224 void processConstructorDeclaration(
225 ConstructorDeclaration node, bool isNew) { 225 ConstructorDeclaration node, bool isNew) {
226 ConstructorElement element = node.element; 226 ConstructorElement element = node.element;
227 if (element != null) { 227 if (element != null) {
228 classElementHolder.addConstructor(element); 228 classElementHolder.addConstructor(element);
229 if (isNew) { 229 if (isNew) {
(...skipping 515 matching lines...) Expand 10 before | Expand all | Expand 10 after
745 } else if (element is ParameterElementImpl) { 745 } else if (element is ParameterElementImpl) {
746 element.setVisibleRange(newOffset, newLength); 746 element.setVisibleRange(newOffset, newLength);
747 } 747 }
748 } 748 }
749 } 749 }
750 } 750 }
751 } 751 }
752 super.visitElement(element); 752 super.visitElement(element);
753 } 753 }
754 } 754 }
OLDNEW
« no previous file with comments | « pkg/analyzer/lib/src/generated/resolver.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