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

Unified Diff: pkg/analysis_server/lib/src/computer/computer_overrides.dart

Issue 479683005: Make more use of generated code in analysis server. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_server/lib/src/computer/computer_overrides.dart
diff --git a/pkg/analysis_server/lib/src/computer/computer_overrides.dart b/pkg/analysis_server/lib/src/computer/computer_overrides.dart
index 52ac7863736cc9abfeb1acd860f8cf2f673d1ea0..fc36d37fb2e38dc902b3a7f91e7a5435fe01fb23 100644
--- a/pkg/analysis_server/lib/src/computer/computer_overrides.dart
+++ b/pkg/analysis_server/lib/src/computer/computer_overrides.dart
@@ -4,9 +4,7 @@
library computer.overrides;
-import 'package:analysis_server/src/computer/element.dart';
-import 'package:analysis_server/src/constants.dart';
-import 'package:analysis_server/src/services/json.dart';
+import 'package:analysis_server/src/protocol2.dart';
import 'package:analyzer/src/generated/ast.dart';
import 'package:analyzer/src/generated/element.dart' as engine;
@@ -68,12 +66,15 @@ class DartUnitOverridesComputer {
// is there any override?
if (superEngineElement != null || interfaceEngineElements.isNotEmpty) {
OverriddenMember superMember = superEngineElement != null ?
- OverriddenMember.fromEngine(superEngineElement) :
+ new OverriddenMember.fromEngine(superEngineElement) :
null;
List<OverriddenMember> interfaceMembers =
- interfaceEngineElements.map(OverriddenMember.fromEngine).toList();
+ interfaceEngineElements.map((engine.Element member) =>
+ new OverriddenMember.fromEngine(member)).toList();
_overrides.add(
- new Override(offset, length, superMember, interfaceMembers));
+ new Override(offset, length, superclassMember: superMember,
+ interfaceMembers: interfaceMembers.isNotEmpty ?
+ interfaceMembers : null));
}
}
@@ -102,84 +103,3 @@ class DartUnitOverridesComputer {
return null;
}
}
-
-
-class OverriddenMember implements HasToJson {
- final Element element;
- final String className;
-
- OverriddenMember(this.element, this.className);
-
- Map<String, Object> toJson() {
- return {
- ELEMENT: element.toJson(),
- CLASS_NAME: className
- };
- }
-
- @override
- String toString() => toJson().toString();
-
- static OverriddenMember fromEngine(engine.Element member) {
- Element element = new Element.fromEngine(member);
- String className = member.enclosingElement.displayName;
- return new OverriddenMember(element, className);
- }
-
- static OverriddenMember fromJson(Map<String, Object> json) {
- Map<String, Object> elementJson = json[ELEMENT];
- Element element = new Element.fromJson(elementJson);
- String className = json[CLASS_NAME];
- return new OverriddenMember(element, className);
- }
-}
-
-
-class Override implements HasToJson {
- final int offset;
- final int length;
- final OverriddenMember superclassMember;
- final List<OverriddenMember> interfaceMembers;
-
- Override(this.offset, this.length, this.superclassMember,
- this.interfaceMembers);
-
- Map<String, Object> toJson() {
- Map<String, Object> json = <String, Object>{};
- json[OFFSET] = offset;
- json[LENGTH] = length;
- if (superclassMember != null) {
- json[SUPER_CLASS_MEMBER] = superclassMember.toJson();
- }
- if (interfaceMembers != null && interfaceMembers.isNotEmpty) {
- json[INTERFACE_MEMBERS] = objectToJson(interfaceMembers);
- }
- return json;
- }
-
- @override
- String toString() => toJson().toString();
-
- static Override fromJson(Map<String, Object> map) {
- int offset = map[OFFSET];
- int length = map[LENGTH];
- // super
- OverriddenMember superclassMember = null;
- {
- Map<String, Object> superJson = map[SUPER_CLASS_MEMBER];
- if (superJson != null) {
- superclassMember = OverriddenMember.fromJson(superJson);
- }
- }
- // interfaces
- List<OverriddenMember> interfaceElements = null;
- {
- List<Map<String, Object>> jsonList = map[INTERFACE_MEMBERS];
- if (jsonList != null) {
- interfaceElements = jsonList.map(OverriddenMember.fromJson).toList();
- }
- }
- // done
- return new Override(offset, length, superclassMember, interfaceElements);
- }
-}

Powered by Google App Engine
This is Rietveld 408576698