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

Issue 14775002: Add LibraryDependencyMirror+CombinatorMirror to source mirrors. (Closed)

Created:
7 years, 7 months ago by Johnni Winther
Modified:
7 years, 1 month ago
Reviewers:
Alan Knight
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Add LibraryDependencyMirror+CombinatorMirror to source mirrors.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+115 lines, -0 lines) Patch
M sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart View 2 chunks +85 lines, -0 lines 0 comments Download
M sdk/lib/_internal/compiler/implementation/mirrors/mirrors.dart View 1 chunk +27 lines, -0 lines 0 comments Download
M sdk/lib/_internal/compiler/implementation/tree/nodes.dart View 2 chunks +3 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Alan Knight
Shouldn't these also be in the Mirror API in general? It's a significant lack there.
7 years, 7 months ago (2013-05-01 17:55:28 UTC) #1
Johnni Winther
7 years, 7 months ago (2013-05-01 17:57:34 UTC) #2
On 2013/05/01 17:55:28, Alan Knight wrote:
> Shouldn't these also be in the Mirror API in general? It's a significant lack
> there.

The source mirrors (in sdk/lib/_internal/compiler/implementation/mirrors/) is a
prototype for dart:mirrors. The two will merge before 1.0.

Powered by Google App Engine
This is Rietveld 408576698