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

[Enhancement] Add a ParameterTypes property to ClrMethod #68

Closed
Fread75 opened this issue Mar 13, 2017 · 2 comments
Closed

[Enhancement] Add a ParameterTypes property to ClrMethod #68

Fread75 opened this issue Mar 13, 2017 · 2 comments

Comments

@Fread75
Copy link
Contributor

Fread75 commented Mar 13, 2017

A simple property returning an array of string representing the types of the parameters of the method.

@Fread75
Copy link
Contributor Author

Fread75 commented Mar 13, 2017

Hello,

I am currently using ClrMd for a professional project, and my company doesn't allow us yet to contribute to open source projects (mostly for legal reasons). They are working on it and in a few months, I would be authorized to submit a pull request to your repo.

In the mean time, I am working on a local clone of your repository. And I create here all the tickets that correspond to the commit I have made on my local repo, mostly to have your feedback in those points, and also in case that you want to correct them directly.

Regards,
Fred

@leculver
Copy link
Contributor

Thank you for the suggestion!

I don't think this is something I want to add to ClrMD. I don't want to expose an API to produce string names of types here. The only way I would implement this API is if I could hand out the real ClrType instances themselves...and that isn't possible with the underlying Dac APIs I use.

You should be able to do this yourself by using IMetaDataImport to walk all of the parameters and do this yourself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants