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

VariableDefinition #26

Closed
dashne opened this issue Jan 6, 2020 · 3 comments
Closed

VariableDefinition #26

dashne opened this issue Jan 6, 2020 · 3 comments
Labels

Comments

@dashne
Copy link

dashne commented Jan 6, 2020

why there is no VariableDefinition like ParameterDefinition,MethodDefinition,..?

@Washi1337
Copy link
Owner

The main reasons are the following:

  1. ParameterDefinition, MethodDefinition and the like are definitions found in the metadata tables, and they follow the names as described in the ECMA-335 specification. Local variables are not stored in the metadata tables, but are rather stored as blob signatures. For this reason, it follows the same naming convention as other blob signatures (VariableSignature).
  2. AsmResolver does not have PDB symbol read support. Therefore, AsmResolver would never be able to assign names to variables.

@dashne
Copy link
Author

dashne commented Jan 7, 2020

Is there any plan for adding PDB symbol read support to AsmResolver ?

@Washi1337
Copy link
Owner

Washi1337 commented Jan 7, 2020

Not in AsmResolver 3 at least. It is on the todo list for 4, but does not have a high priority right now.

Keep in mind this does not mean AsmResolver is not able to read the local variables from a method body.

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

No branches or pull requests

2 participants