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

Issue 10911344: Fix for issue #5227: do not declare type variables in the closure analyzer when they are not needed. (Closed)

Created:
8 years, 3 months ago by ngeoffray
Modified:
8 years, 3 months ago
Reviewers:
karlklose, kasperl
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix for issue #5227: do not declare type variables in the closure analyzer when they are not needed. Committed: https://code.google.com/p/dart/source/detail?r=12469

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -6 lines) Patch
M lib/compiler/implementation/closure.dart View 2 chunks +9 lines, -6 lines 0 comments Download
A tests/language/closure_in_constructor_test.dart View 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ngeoffray
8 years, 3 months ago (2012-09-17 21:32:26 UTC) #1
kasperl
8 years, 3 months ago (2012-09-18 05:58:44 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698