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

Unified Diff: pkg/analysis_server/lib/src/services/completion/suggestion_builder.dart

Issue 1260593005: update suggestion element return type to have type param (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 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/services/completion/suggestion_builder.dart
diff --git a/pkg/analysis_server/lib/src/services/completion/suggestion_builder.dart b/pkg/analysis_server/lib/src/services/completion/suggestion_builder.dart
index e1132b537a53b83b54f8cae26f2534f80fd3e1fd..c6e376a2c91ab8f453522dda00d922885d6bbed8 100644
--- a/pkg/analysis_server/lib/src/services/completion/suggestion_builder.dart
+++ b/pkg/analysis_server/lib/src/services/completion/suggestion_builder.dart
@@ -28,35 +28,10 @@ const String DYNAMIC = 'dynamic';
CompletionSuggestion createSuggestion(Element element,
{CompletionSuggestionKind kind: CompletionSuggestionKind.INVOCATION,
int relevance: DART_RELEVANCE_DEFAULT, Source importForSource}) {
- String nameForType(DartType type) {
- if (type == null) {
- return DYNAMIC;
- }
- String name = type.displayName;
- if (name == null || name.length <= 0) {
- return DYNAMIC;
- }
- //TODO (danrubel) include type arguments ??
- return name;
- }
-
- String returnType = null;
- if (element is ExecutableElement) {
- if (element.isOperator) {
- // Do not include operators in suggestions
- return null;
- }
- if (element is PropertyAccessorElement && element.isSetter) {
- // no return type
- } else {
- returnType = nameForType(element.returnType);
- }
- } else if (element is VariableElement) {
- returnType = nameForType(element.type);
- } else if (element is FunctionTypeAliasElement) {
- returnType = nameForType(element.returnType);
+ if (element is ExecutableElement && element.isOperator) {
+ // Do not include operators in suggestions
+ return null;
}
-
String completion = element.displayName;
bool isDeprecated = element.isDeprecated;
CompletionSuggestion suggestion = new CompletionSuggestion(kind,
@@ -67,7 +42,7 @@ CompletionSuggestion createSuggestion(Element element,
if (enclosingElement is ClassElement) {
suggestion.declaringType = enclosingElement.displayName;
}
- suggestion.returnType = returnType;
+ suggestion.returnType = getReturnTypeString(element);
if (element is ExecutableElement && element is! PropertyAccessorElement) {
suggestion.parameterNames = element.parameters
.map((ParameterElement parameter) => parameter.name)

Powered by Google App Engine
This is Rietveld 408576698