Skip to content

Conversation

@lewing
Copy link
Member

@lewing lewing commented Nov 10, 2021

Rework MonoSdbHelper and tie its lifetime to the ExecutionContext to simplify session handling.

@ghost ghost added the area-Debugger-mono label Nov 10, 2021
@ghost
Copy link

ghost commented Nov 10, 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 added the arch-wasm WebAssembly architecture label Nov 10, 2021
@ghost
Copy link

ghost commented Nov 10, 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
Copy link
Member Author

lewing commented Nov 10, 2021

@thaystg the smaller version is smaller but...

@lewing lewing force-pushed the binary-writer branch 4 times, most recently from 214efa8 to ca4de82 Compare November 11, 2021 18:07
@lewing lewing marked this pull request as ready for review November 11, 2021 18:57
@lewing lewing requested a review from marek-safar as a code owner November 11, 2021 18:57
@lewing lewing marked this pull request as draft November 15, 2021 20:45
@lewing lewing changed the title [wasm][debugger] Rework memory stream management [wasm][debugger] Tie sdb agent lifetime to the ExecutionContext and simplify the api Nov 16, 2021
@lewing lewing requested a review from thaystg November 16, 2021 18:18
@lewing lewing marked this pull request as ready for review November 17, 2021 17:18
@lewing
Copy link
Member Author

lewing commented Nov 17, 2021

Needs a little more work, the merge was slightly off. Working on it now

@lewing
Copy link
Member Author

lewing commented Nov 18, 2021

should be good now.

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.

3 participants