Skip to content

Commit 2112d50

Browse files
committed
Clarify the handlers
1 parent e65b4f5 commit 2112d50

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/features/DebugSession.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,10 +151,12 @@ export class DebugSessionFeature extends LanguageClientConsumer
151151
}
152152

153153

154-
// The dispose shorthand demonry for making an event one-time courtesy of: https://github.com/OmniSharp/omnisharp-vscode/blob/b8b07bb12557b4400198895f82a94895cb90c461/test/integrationTests/launchConfiguration.integration.test.ts#L41-L45
155154

155+
// Ensure the .NET session stops before the PowerShell session so that the .NET debug session doesnt emit an error about the process unexpectedly terminating
156156
const startDebugEvent = vscode.debug.onDidStartDebugSession((dotnetAttachSession) => {
157157
// Makes the event one-time
158+
// HACK: This seems like you would be calling a method on a variable not assigned yet, but it does work in the flow.
159+
// The dispose shorthand demonry for making an event one-time courtesy of: https://github.com/OmniSharp/omnisharp-vscode/blob/b8b07bb12557b4400198895f82a94895cb90c461/test/integrationTests/launchConfiguration.integration.test.ts#L41-L45
158160
startDebugEvent.dispose();
159161
this.logger.write(`Debugger session detected: ${dotnetAttachSession.name} (${dotnetAttachSession.id})`);
160162
if (dotnetAttachSession.configuration.name == dotnetDebuggerConfig.name) {

0 commit comments

Comments
 (0)