-
Notifications
You must be signed in to change notification settings - Fork 1.7k
dart2js: should provide a complete implementation of dart:mirrors at runtime #6490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Issue #6907 has been merged into this issue. cc @floitschG. |
Set owner to @peter-ahe-google. |
The following should work now (in unminified code): * Invoking instance methods. * Setting instane fields. * Getting instance fields. I'm currently working on implementing support for reflecting on libraries. |
Added TriageForM5 label. |
https://codereview.chromium.org/15858005/ provides support setting and getting fields in library and class mirrors. For those who are counting, this means that 4 out of 10 tests are passing in https://code.google.com/p/dart/source/browse/branches/bleeding_edge/dart/tests/lib/mirrors/mirrors_test.dart Remember that this is not yet supported in minified code. |
CL 15858005 landed in r23070. |
https://codereview.chromium.org/15895002/ provides support for ClosureMirror. For those who are counting, this means that 5 out of 10 tests are passing in tests/lib/mirrors/mirrors_test.dart. Still no support for minified code. |
https://codereview.chromium.org/15857005/ implements newInstance on ClassMirror. For those who are counting, this means that 6 out of 10 tests are passing in tests/lib/mirrors/mirrors_test.dart. Still no support for minified code. |
CL 15895002 committed as r23135. |
https://codereview.chromium.org/15861005/ implements ClassMirror.owner and LibraryMirror.uri. For those who are counting, this means that 8 out of 10 tests are passing in tests/lib/mirrors/mirrors_test.dart. Still no support for minified code. |
CL 15861005 committed as r23211. |
https://codereview.chromium.org/16097002/ implements simple and qualified names. For those who are counting, this means that 9 out of 10 tests are passing in tests/lib/mirrors/mirrors_test.dart. Still no support for minified code. |
CL 16097002 committed as r23225. |
Issue #10905 tracks providing reflective access to metadata in dart2js. |
Removed TriageForM5 label. |
https://codereview.chromium.org/15979020/ implements InstanceMirror.invoke in minified code. |
CL 15979020 landed in r23596. |
we are now tracking missing features on individual bugs, so we can close this one. |
In r14469 we stubbed out some of how runtime mirrors can work in dart2js.
There are great technical challenges to overcome in implementing mirrors for dart2js, and we expect there wil be a non-trivial overhead for the most general usages.
At the time of writing, mirrors for are not being actively developed as the dart2js team is focusing on performance and language feature completeness. But patches are welcome!
The text was updated successfully, but these errors were encountered: