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

Issue 9619009: A local may be an unnamed closure. Make sure we emit a name for them. (Closed)

Created:
8 years, 9 months ago by ngeoffray
Modified:
8 years, 9 months ago
Reviewers:
ahe, floitsch
CC:
reviews_dartlang.org, ahe, karlklose, Lasse Reichstein Nielsen, kasperl
Visibility:
Public.

Description

A local may be an unnamed closure. Make sure we emit a name for them. Committed: https://code.google.com/p/dart/source/detail?r=5102

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -7 lines) Patch
M client/tests/client/client-leg.status View 1 chunk +0 lines, -4 lines 0 comments Download
M frog/leg/scanner/keyword.dart View 1 chunk +1 line, -1 line 0 comments Download
M frog/leg/ssa/codegen.dart View 2 chunks +2 lines, -2 lines 0 comments Download
A tests/language/src/UnnamedClosureTest.dart View 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
ngeoffray
8 years, 9 months ago (2012-03-07 15:57:01 UTC) #1
ahe
LGTM
8 years, 9 months ago (2012-03-07 16:09:54 UTC) #2
ngeoffray
8 years, 9 months ago (2012-03-07 16:16:56 UTC) #3
Thanks Peter

Powered by Google App Engine
This is Rietveld 408576698