Skip to content

Conversation

@mikaelwaltersson
Copy link
Contributor

This PR includes bug fixes for the following issues:

  • When debugging WebAssembly modules if you use the Watch or REPL to view values only the top level value will be shown, any value with children will not be viewable.
  • The memory viewer does not work if the WebAssembly memory is backed by a SharedArrayBuffer

Screenshot of current behavior using the REPL to view a variable with children:
c-wasm-app-debugging-not-working

Screenshot of behavior when running this PR version:
c-wasm-app-debugging

Example code used in the the screenshots:
c-wasm-app.zip

@mikaelwaltersson
Copy link
Contributor Author

@mikaelwaltersson please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

Copy link
Member

@connor4312 connor4312 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thank you! 👏

@connor4312 connor4312 enabled auto-merge (squash) April 8, 2025 16:52
@vs-code-engineering vs-code-engineering bot added this to the April 2025 milestone Apr 8, 2025
@connor4312 connor4312 merged commit 1942fd6 into microsoft:main Apr 8, 2025
5 of 7 checks passed
@mikaelwaltersson mikaelwaltersson deleted the fix-floating-wasm-variable branch April 8, 2025 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants