Skip to content

Add EvaluationResult Scheme for arrays. - #67095

Merged
ilonatommy merged 8 commits into
dotnet:mainfrom
ilonatommy:add-evaluation-result-scheme
May 26, 2022
Merged

Add EvaluationResult Scheme for arrays.#67095
ilonatommy merged 8 commits into
dotnet:mainfrom
ilonatommy:add-evaluation-result-scheme

Conversation

@ilonatommy

@ilonatommy ilonatommy commented Mar 24, 2022

Copy link
Copy Markdown
Member

Requires merging in #66654 to work.

Objects returned by a method executed on a primitive type do not have an objectId, they don't have their representation on the runtime side either. To be able to get their properties (e.g. elements of array) we are cacheing them on the debugger side and granting debugger-side-objectId in this PR.

@ilonatommy
ilonatommy requested a review from radical March 24, 2022 12:46
@ilonatommy
ilonatommy requested a review from marek-safar as a code owner March 24, 2022 12:46
@ilonatommy ilonatommy self-assigned this Mar 24, 2022
@ilonatommy ilonatommy added arch-wasm WebAssembly architecture area-Debugger-mono labels Mar 24, 2022
@ghost

ghost commented Mar 24, 2022

Copy link
Copy Markdown

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Requires merging in #66654 to work.

Objects returned by a method executed on a primitive type do not have an objectId, they don't have their representation on the runtime side either. To be able to get their properties (e.g. elements of array) we are caching them on the debugger side and granting debugger-side-objectId in this PR.

Author: ilonatommy
Assignees: ilonatommy
Labels:

arch-wasm, area-Debugger-mono

Milestone: -

@ilonatommy
ilonatommy requested a review from thaystg May 23, 2022 12:54
@ilonatommy
ilonatommy merged commit 4e9ac49 into dotnet:main May 26, 2022
ilonatommy added a commit to ilonatommy/runtime that referenced this pull request Jun 10, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jun 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-wasm WebAssembly architecture area-Debugger-mono

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants