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

Issue 2476613002: Remove generalized tear-offs from the specification. (Closed)

Created:
4 years, 1 month ago by Lasse Reichstein Nielsen
Modified:
4 years, 1 month ago
Reviewers:
eernst
CC:
reviews_dartlang.org, floitsch, mit
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove generalized tear-offs from the specification. Fixes issue #27519 BUG= http://dartbug.com/27519 R=eernst@google.com Committed: https://github.com/dart-lang/sdk/commit/4cb61407a8bbd6050e876995f149b07cf9299445

Patch Set 1 #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -231 lines) Patch
M docs/language/dartLangSpec.tex View 10 chunks +15 lines, -231 lines 6 comments Download

Messages

Total messages: 7 (2 generated)
Lasse Reichstein Nielsen
4 years, 1 month ago (2016-11-03 11:38:58 UTC) #2
eernst
LGTM. I added a comment on the missing argument types in the closure created for ...
4 years, 1 month ago (2016-11-03 13:13:46 UTC) #3
Lasse Reichstein Nielsen
https://codereview.chromium.org/2476613002/diff/1/docs/language/dartLangSpec.tex File docs/language/dartLangSpec.tex (right): https://codereview.chromium.org/2476613002/diff/1/docs/language/dartLangSpec.tex#newcode4038 docs/language/dartLangSpec.tex:4038: Unconditional property extraction has one of two syntactic forms: ...
4 years, 1 month ago (2016-11-07 07:49:06 UTC) #4
eernst
lgtm https://codereview.chromium.org/2476613002/diff/1/docs/language/dartLangSpec.tex File docs/language/dartLangSpec.tex (right): https://codereview.chromium.org/2476613002/diff/1/docs/language/dartLangSpec.tex#newcode4038 docs/language/dartLangSpec.tex:4038: Unconditional property extraction has one of two syntactic ...
4 years, 1 month ago (2016-11-07 10:08:40 UTC) #5
Lasse Reichstein Nielsen
4 years, 1 month ago (2016-11-07 13:54:06 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
4cb61407a8bbd6050e876995f149b07cf9299445 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698