Merge pull request #665 from fortran-lang/bug/debug

Improvements on Run and Debug buttons

Author: gnikit

package.json

@@ -628,7 +628,7 @@
           "command": "fortran.build.debug",
           "group": "navigation@1",
           "title": "Debug Fortran File",
-          "when": "resourceLangId == fortran && !isInDiffEditor && !virtualWorkspace && shellExecutionSupported"
+          "when": "resourceLangId =~ /Fortran/ && !isInDiffEditor && !virtualWorkspace && shellExecutionSupported"
         }
       ]
     }

src/features/linter-provider.ts

@@ -296,7 +296,7 @@ export class FortranLintingProvider {
         program: `${textDocument.fileName}.o`,
         cwd: folder.uri.fsPath,
       };
-      await vscode.debug.startDebugging(folder, selectedConfig, { noDebug: debug });
+      await vscode.debug.startDebugging(folder, selectedConfig, { noDebug: !debug });
       return;
     } catch (err) {
       this.logger.error(`[build] Compiling ${textDocument.fileName} failed:`, err);
@@ -417,7 +417,8 @@ export class FortranLintingProvider {
    */
   private getLinterExtraArgs(): string[] {
     const config = vscode.workspace.getConfiguration(EXTENSION_ID);
-    let args: string[] = this.linter.argsDefault;
+    // Get the linter arguments from the settings via a deep copy
+    let args: string[] = [...this.linter.argsDefault];
     const user_args: string[] = this.settings.args;
     // If we have specified linter.extraArgs then replace default arguments
     if (user_args.length > 0) args = user_args.slice();

src/lib/linters.ts

@@ -14,28 +14,28 @@ abstract class Linter {
     /**
      * Linter name
      */
-    public name: string,
+    public readonly name: string,
     /**
      * Regular expression to match compiler messages
      */
-    public regex: RegExp,
+    public readonly regex: RegExp,
     /**
      * Keywords used to identify message severity
      */
-    public severity: LinterSeverityKeywords,
+    public readonly severity: LinterSeverityKeywords,
     /**
      * Mandatory linter arguments. These are always present
      */
-    public args: string[],
+    public readonly args: string[],
     /**
      * Command line arguments to pass to the linter, these get overwritten in
      * case the user has set arguments in the settings
      */
-    public argsDefault?: string[],
+    public readonly argsDefault?: string[],
     /**
      * Compiler flag used to change the directory output for modules
      */
-    public modFlag?: string
+    public readonly modFlag?: string
   ) {}
 
   public getSeverityLevel(msg_type: string): vscode.DiagnosticSeverity {