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

[wasm][debugger] Correct the endian swapping and string implementation #61386

Merged
merged 5 commits into from
Nov 10, 2021

Conversation

lewing
Copy link
Member

@lewing lewing commented Nov 9, 2021

A limited change fix WriteString and only byte swap when needed.

@ghost
Copy link

ghost commented Nov 9, 2021

Tagging subscribers to this area: @thaystg
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: lewing
Assignees: -
Labels:

area-Debugger-mono

Milestone: -

@lewing lewing changed the title Correct the endian swapping and string implementation [wasm][debugger] Correct the endian swapping and string implementation Nov 9, 2021
@lewing lewing added the arch-wasm WebAssembly architecture label Nov 9, 2021
@ghost
Copy link

ghost commented Nov 9, 2021

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

Issue Details

null

Author: lewing
Assignees: -
Labels:

arch-wasm, area-Debugger-mono

Milestone: -

@lewing lewing requested a review from radical November 9, 2021 23:27
@lewing lewing marked this pull request as ready for review November 10, 2021 02:09
@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2021
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.

4 participants