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

EE doesn't display ref fields as expected. #62157

Open
AaronRobinsonMSFT opened this issue Jun 27, 2022 · 0 comments
Open

EE doesn't display ref fields as expected. #62157

AaronRobinsonMSFT opened this issue Jun 27, 2022 · 0 comments

Comments

@AaronRobinsonMSFT
Copy link
Member

AaronRobinsonMSFT commented Jun 27, 2022

The following code should be updated for the new ref field support. See #62155.

Note the RStr field in the example below is a ref field. I would expect the display to be either ref string or to point through the ref and display the value.

image

public Type Type
{
get
{
var t = ClrType?.GetLmrType();
//TODO: Sometimes we get byref types here when dealing with ref-returning members.
// That probably should not happen.
// For now we will just unwrap unexpected byrefs
if (t?.IsByRef == true)
{
t = t.GetElementType();
Debug.Assert(!t.IsByRef, "double byref type?");
}
return t;
}
}

Relates to test plan #59194

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Jun 27, 2022
@vatsalyaagrawal vatsalyaagrawal added Interactive-Debugging Area-Interactive Concept-Design Debt Engineering Debt, Design Debt, or poor product code quality and removed untriaged Issues and PRs which have not yet been triaged by a lead Area-IDE labels Jun 27, 2022
@vatsalyaagrawal vatsalyaagrawal added this to the Backlog milestone Jun 27, 2022
@tmat tmat assigned cston and unassigned tmat Jun 27, 2022
@jcouv jcouv added the Bug label Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants