fix problems found in review

WebFreak001 · Mar 10, 2024 · 5e9abd8 · 5e9abd8
1 parent ec2a93a
commit 5e9abd8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/backend/mi2/mi2.ts b/src/backend/mi2/mi2.ts
@@ -315,7 +315,7 @@ export class MI2 extends EventEmitter implements IBackend {
 	}
 
 	onOutputStderr(str: string) {
-		let lines = str.split('\n');
+		const lines = str.split('\n');
 		lines.forEach(line => {
 			this.log("stderr", line);
 		});
@@ -330,7 +330,7 @@ export class MI2 extends EventEmitter implements IBackend {
 	}
 
 	onOutput(str: string) {
-		let lines = str.split('\n');
+		const lines = str.split('\n');
 		lines.forEach(line => {
 			if (couldBeOutput(line)) {
 				if (!gdbMatch.exec(line))
@@ -835,7 +835,7 @@ export class MI2 extends EventEmitter implements IBackend {
 			this.log("stderr", "varCreate");
 		let miCommand = "var-create ";
 		if (threadId != 0) {
-			miCommand += `--thread ${threadId} --frame ${frameLevel}`
+			miCommand += `--thread ${threadId} --frame ${frameLevel}`;
 		}
 		const res = await this.sendCommand(`${miCommand} ${this.quote(name)} ${frame} "${expression}"`);
 		return new VariableObject(res.result(""));

diff --git a/src/mibase.ts b/src/mibase.ts
@@ -285,7 +285,8 @@ export class MI2DebugSession extends DebugSession {
 			this.sendResponse(response);
 		}).catch((error: MIError) => {
 			if (error.message === 'Selected thread is running.') {
-				console.error(error);
+				console.error(error.message);
+				this.sendResponse(response);
 				return;
 			}
 			this.sendErrorResponse(response, 17, `Could not get threads: ${error}`);