-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[release/8.0-staging][mono][debugger] Fix inspect Span<byte> #99844
Conversation
Tagging subscribers to this area: @thaystg |
@thaystg looks like the bot didn't put the backport template in the description for some reason |
This is because I did it manually. I will add the template and fill it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved. we will take for consideration in 8.0.x
@jeffschwMSFT I'd sure love for this to be taken. It's completely blocking testing an android app that I've been (personally) developing. |
it is approved for 8.0.5 shipping in May |
Backport of #99843 to release/8.0-staging
Customer Impact
Reported by customers in dotnet/android#8792. While debugging a MAUI/Blazor Wasm app and it's paused in a breakpoint/stepping and there is any local that is of type Span the runtime will throw an assertion and finish the debugging session.
Regression
Testing
Risk
Low. The change only checks if the value that is coming from SDB is NULL and then do not throw the assertion.