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

Unified Diff: pkg/front_end/lib/src/fasta/kernel/body_builder.dart

Issue 2916533003: Set return type of factory constructors properly. (Closed)
Patch Set: Created 3 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/front_end/lib/src/fasta/kernel/body_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/body_builder.dart b/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
index bd03e5fe879fc96702b3fe122cdd1a1f17ae8e27..db00c358d2976b75826e10468100d10dc689d1e2 100644
--- a/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/body_builder.dart
@@ -482,9 +482,6 @@ class BodyBuilder extends ScopeListener<JumpTarget> implements BuilderHelper {
DartType _computeReturnTypeContext(MemberBuilder member) {
if (member is KernelProcedureBuilder) {
- if (member.target.kind == ProcedureKind.Factory) {
- return computeFactoryConstructorReturnType(member.target);
- }
ahe 2017/06/01 09:10:08 I have to "steal" this :-)
return member.target.function.returnType;
} else {
assert(member is KernelConstructorBuilder);

Powered by Google App Engine
This is Rietveld 408576698