Merge branch 'master' into static_variable

Author: HenryRiley0

CHANGELOG.md

@@ -13,6 +13,10 @@ Versioning].
 ### Added
 
 - add static vairable support ([@henryriley0])
+- fix gdb check error when debug beginning ([@henryriley0])
+- fix implicitly type error in log message when build vsix ([@henryriley0])
+- check for configured debugger before start to provide a nicer error message
+  ([@GitMensch])
 
 ## [0.27.0] - 2024-02-07
 
@@ -21,6 +25,8 @@ Versioning].
 - Added registers view ([@nomtats]) #242
 - Enabled breakpoints inside `riscv` files ([@William-An]) #404
 
+[0.27.0]: https://github.com/WebFreak001/code-debug/compare/v0.26.1...v0.27.0
+
 ## [0.26.1] - 2022-12-31
 
 ### Fixed

README.md

@@ -213,4 +213,10 @@ differently based on whether the remote system is a POSIX or a Windows system.
 You may need to experiment to find the correct escaping necessary for the command to be
 sent to the debugger as you intended.
 
+### LogMessage
+
+LogMessage will print a message in the debug console when breakpoint is hit. Expressions within {} are interpolated.
+
+![LogMessage](images/logMessage.gif)
+
 ## [Issues](https://github.com/WebFreak001/code-debug)

images/logMessage.gif

@@ -10,7 +10,7 @@
 		"debug"
 	],
 	"license": "public domain",
-	"version": "0.27.0",
+	"version": "0.27.1",
 	"publisher": "webfreak",
 	"icon": "images/icon.png",
 	"engines": {
@@ -187,7 +187,7 @@
 							"valuesFormatting": {
 								"type": "string",
 								"description": "Set the way of showing variable values. 'disabled' - show value as is, 'parseText' - parse debuggers output text into structure, 'prettyPrinters' - enable debuggers custom pretty-printers if there are any",
-								"default": "parseText",
+								"default": "prettyPrinters",
 								"enum": [
 									"disabled",
 									"parseText",
@@ -1112,6 +1112,7 @@
 		"@istanbuljs/nyc-config-typescript": "^1.0.2",
 		"@types/mocha": "^5.2.6",
 		"@types/node": "^11.11.3",
+		"@types/ssh2": "^1.15.0",
 		"@types/vscode": "^1.55.0",
 		"@typescript-eslint/eslint-plugin": "^5.22.0",
 		"@typescript-eslint/parser": "^5.22.0",
@@ -1125,7 +1126,7 @@
 		"nyc": "^15.1.0",
 		"prettier": "^2.6.2",
 		"ts-node": "^10.8.0",
-		"typescript": "^3.9.3"
+		"typescript": "^4.3.2"
 	},
 	"__metadata": {
 		"id": "2fd22b8e-b3b8-4e7f-9a28-a5e2d1bdd0d4",

package-lock.json

@@ -4,11 +4,13 @@ import { DebugProtocol } from "vscode-debugprotocol/lib/debugProtocol";
 export type ValuesFormattingMode = "disabled" | "parseText" | "prettyPrinters";
 
 export interface Breakpoint {
+	id?:number;
 	file?: string;
 	line?: number;
 	raw?: string;
 	condition: string;
 	countCondition?: string;
+	logMessage?: string;
 }
 
 export interface Thread {
@@ -60,8 +62,8 @@ export interface IBackend {
 	attach(cwd: string, executable: string, target: string, autorun: string[]): Thenable<any>;
 	connect(cwd: string, executable: string, target: string, autorun: string[]): Thenable<any>;
 	start(runToStart: boolean): Thenable<boolean>;
-	stop();
-	detach();
+	stop(): void;
+	detach(): void;
 	interrupt(): Thenable<boolean>;
 	continue(): Thenable<boolean>;
 	next(): Thenable<boolean>;
@@ -141,29 +143,25 @@ export interface MIError extends Error {
 	readonly source: string;
 }
 export interface MIErrorConstructor {
-	new (message: string, source: string): MIError;
+	new(message: string, source: string): MIError;
 	readonly prototype: MIError;
 }
 
-export const MIError: MIErrorConstructor = <any> class MIError {
-	readonly name: string;
-	readonly message: string;
-	readonly source: string;
+export const MIError: MIErrorConstructor = class MIError {
+	private readonly _message: string;
+	private readonly _source: string;
 	public constructor(message: string, source: string) {
-		Object.defineProperty(this, 'name', {
-			get: () => (this.constructor as any).name,
-		});
-		Object.defineProperty(this, 'message', {
-			get: () => message,
-		});
-		Object.defineProperty(this, 'source', {
-			get: () => source,
-		});
+		this._message = message;
+		this._source = source;
 		Error.captureStackTrace(this, this.constructor);
 	}
 
+	get name() { return this.constructor.name; }
+	get message() { return this._message; }
+	get source() { return this._source; }
+
 	public toString() {
-		return `${this.message} (from ${this.source})`;
+		return `${this.message} (from ${this._source})`;
 	}
 };
 Object.setPrototypeOf(MIError as any, Object.create(Error.prototype));

package.json

@@ -1,3 +1,4 @@
+import { VariableObject } from "./backend";
 import { MINode } from "./mi_parse";
 
 const resultRegex = /(static )?([a-zA-Z_\-][a-zA-Z0-9_\-]*|\[\d+\])\s*=\s*/;
@@ -29,7 +30,7 @@ export function isExpandable(value: string): number {
 	else return 0;
 }
 
-export function expandValue(variableCreate: Function, value: string, root: string = "", extra: any = undefined): any {
+export function expandValue(variableCreate: (arg: VariableObject | string, options?: any) => any, value: string, root: string = "", extra: any = undefined): any {
 	const parseCString = () => {
 		value = value.trim();
 		if (value[0] != '"' && value[0] != '\'')
@@ -56,10 +57,10 @@ export function expandValue(variableCreate: Function, value: string, root: strin
 	};
 
 	const stack = [root];
-	let parseValue, parseCommaResult, parseCommaValue, parseResult, createValue;
+	let parseValue: () => any, parseCommaResult: (pushToStack: boolean) => any, parseCommaValue: () => any, parseResult: (pushToStack: boolean) => any, createValue: (name: string, val: any) => any;
 	let variable = "";
 
-	const getNamespace = (variable) => {
+	const getNamespace = (variable: string) => {
 		let namespace = "";
 		let prefix = "";
 		stack.push(variable);
@@ -209,7 +210,7 @@ export function expandValue(variableCreate: Function, value: string, root: strin
 		return createValue(name, val);
 	};
 
-	createValue = (name, val) => {
+	createValue = (name: string, val: any) => {
 		let ref = 0;
 		if (typeof val == "object") {
 			ref = variableCreate(val);
@@ -223,7 +224,7 @@ export function expandValue(variableCreate: Function, value: string, root: strin
 				val = "Object@" + val;
 			}
 		} else if (typeof val == "string" && val.startsWith("@0x")) {
-			ref = variableCreate(getNamespace("*&" + name.substr));
+			ref = variableCreate(getNamespace("*&" + name.substring(1)));
 			val = "Ref" + val;
 		} else if (typeof val == "string" && val.startsWith("<...>")) {
 			ref = variableCreate(getNamespace(name));