Skip to content

Support GoToTargets Request #1102

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.

Sign up for GitHub

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

Merged
WardenGnaw merged 12 commits into from
Jan 29, 2021
Merged

Support GoToTargets Request #1102

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b874c5c
implement DAP Goto Request
Trass3r Aug 1, 2020
a64c8d5
use List for HandleCollection
Trass3r Sep 9, 2020
1138d4e
use HandleCollection for goto targets
Trass3r Sep 9, 2020
57f5d33
cleanup
Trass3r Sep 11, 2020
9b88764
Revert "use HandleCollection for goto targets"
Trass3r Oct 20, 2020
f119e0c
Fixing todo
WardenGnaw Oct 28, 2020
1ebc16d
add comment about potential race condition
Trass3r Nov 6, 2020
a2558e7
fix exception description so it shows something in vscode
Trass3r Jan 9, 2021
15de662
send "goto" stopped event after sending the command response
Trass3r Jan 9, 2021
f37fe48
Remove Jump for only GoToTarget and add memory ref
WardenGnaw Jan 28, 2021
8fe364a
Addressing PR issues
WardenGnaw Jan 29, 2021
25625cb
Merge branch 'main' into dev/waan/SupportGoToTargets
WardenGnaw Jan 29, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
send "goto" stopped event after sending the command response
  • Loading branch information
@Trass3r @WardenGnaw
Trass3r authored and WardenGnaw committed Jan 28, 2021
commit 15de6625669aad25b03830b5d17f07cfa93868f0
4 changes: 3 additions & 1 deletion src/OpenDebugAD7/AD7DebugSession.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1334,11 +1334,11 @@ protected override void HandleGotoRequestAsync(IRequestResponder<GotoArguments>
}

var builder = new ErrorBuilder(() => AD7Resources.Error_UnableToSetNextStatement);
IDebugThread2 thread = null;
try
{
if (m_gotoCodeContexts.TryGetValue(responder.Arguments.TargetId, out IDebugCodeContext2 gotoTarget))
{
IDebugThread2 thread = null;
lock (m_threads)
{
if (!m_threads.TryGetValue(responder.Arguments.ThreadId, out thread))
Expand All @@ -1352,9 +1352,11 @@ protected override void HandleGotoRequestAsync(IRequestResponder<GotoArguments>
{
m_isStopped = true;
responder.SetError(new ProtocolException(e.Message));
return;
}

responder.SetResponse(response);
FireStoppedEvent(thread, StoppedEvent.ReasonValue.Goto);
}

protected override void HandleGotoTargetsRequestAsync(IRequestResponder<GotoTargetsArguments, GotoTargetsResponse> responder)
Expand Down