Skip to content
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

Jump to bottom

No way to get the qualified name from a Type object #9395

Closed
justinfagnani opened this issue Mar 23, 2013 · 7 comments
Closed

No way to get the qualified name from a Type object #9395

justinfagnani opened this issue Mar 23, 2013 · 7 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue library-mirrors P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@justinfagnani
Copy link
Contributor

Using Type with mirrors is very difficult because of this. You can use a Type object to look up member by simple name, but not qualified name.
@gbracha
Copy link
Contributor

gbracha commented Mar 25, 2013

Lets discuss.

Set owner to @gbracha.
Added Accepted label.

@sethladd
Copy link
Contributor

sethladd commented Apr 8, 2013

Added C1 label.

@jmesserly
Copy link

@justin -- given a Type, you can now call reflectClass and get the TypeMirror, which has a qualified name. Does that work?

@jmesserly
Copy link

Removed Priority-Medium label.
Added Priority-High label.

@DartBot
Copy link

DartBot commented Apr 19, 2013

This comment was originally written by voj...@google.com

Yep, I think reflectClass solves this. Btw. I would call it reflectType(),
just my $0.02

@sethladd
Copy link
Contributor

For those of you following along at home, here's the docs: http://api.dartlang.org/docs/bleeding_edge/dart_mirrors.html#reflectClass

(Apparently it's not in a trunk build just yet)

@gbracha
Copy link
Contributor

gbracha commented Apr 22, 2013

Lowering the priority since reflectClass addresses the issue. There is some question of whether we really want Type to do this. Type is intended as a key to the mirror system, not as a reflective object in itself.

Removed Priority-High label.
Added Priority-Medium label.

@justinfagnani justinfagnani added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-mirrors labels Apr 22, 2013
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed accepted labels Feb 29, 2016
@munificent munificent removed the C1 label Jun 20, 2018
@a-siva a-siva added the closed-not-planned Closed as we don't intend to take action on the reported issue label Aug 5, 2022
@a-siva a-siva closed this as completed Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue library-mirrors P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@sethladd @kevmoo @munificent @justinfagnani @jmesserly @gbracha @a-siva @DartBot