20408: Corrected debugger line numbers for non-versioned Amalgam buil…

…ds (#13) If the Amalgam version is `0.0.0` use 1-based line numbers instead of 0-based.
howsoai · May 28, 2024 · ba2743b · ba2743b
1 parent df455c6
commit ba2743b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "amalgam-lang",
-    "version": "1.2.0",
+    "version": "1.2.1",
     "type": "commonjs",
     "publisher": "howso",
     "displayName": "Amalgam Language",

diff --git a/src/debugger/session.ts b/src/debugger/session.ts
@@ -503,7 +503,7 @@ export class AmalgamDebugSession extends LoggingDebugSession {
     try {
       const executableVersion = await this.getExecutableVersion(executable);
       outputLogger.info(`Amalgam Version: ${executableVersion}`);
-      if (semver.lt(executableVersion, "50.0.2")) {
+      if (semver.gt(executableVersion, "0.0.0") && semver.lt(executableVersion, "50.0.2")) {
         this.setDebuggerLinesStartAt1(false);
         this.setDebuggerColumnsStartAt1(false);
         outputLogger.debug("Line numbers set to 0-based");