Skip to content

Commit 92d0fd7

Browse files
committed
Fixes parsing Error level issues with NAG linter
1 parent 3e39eb2 commit 92d0fd7

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/features/linter-provider.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,6 @@ export class FortranLintingProvider {
414414
case 'panic':
415415
case 'fatal':
416416
case 'error':
417-
case 'fatal error':
418417
severity = vscode.DiagnosticSeverity.Error;
419418
break;
420419

@@ -431,9 +430,10 @@ export class FortranLintingProvider {
431430
severity = vscode.DiagnosticSeverity.Information;
432431
break;
433432

433+
// fatal error, sequence error, etc.
434434
default:
435435
severity = vscode.DiagnosticSeverity.Error;
436-
console.log('Unknown severity: ' + msg_type);
436+
console.log('Using default Error Severity for: ' + msg_type);
437437
break;
438438
}
439439

@@ -486,8 +486,13 @@ export class FortranLintingProvider {
486486
// see https://regex101.com/r/GZ0Lzz/2
487487
return /^(?<fname>(?:\w:\\)?.*)\((?<ln>\d+)\):\s*(?:#(?:(?<sev2>\w*):\s*(?<msg2>.*$))|(?<sev1>\w*)\s*(?<msg1>.*$)(?:\s*.*\s*)(?<cn>-*\^))/gm;
488488

489+
/*
490+
See Section 7 of the NAGFOR manual, although it is not accurate with regards
491+
to all the possible messages.
492+
severity: filename, line No.: message
493+
*/
489494
case 'nagfor':
490-
return /^(?<sev1>Remark|Info|Note|Warning|Questionable|Extension|Deleted feature used|Error|Fatal(?: Error)?|Panic)(\(\w+\))?: (?<fname>[\S ]+), line (?<ln>\d+): (?<msg1>.+)$/gm;
495+
return /^(?<sev1>Remark|Info|Note|Warning|Questionable|Extension|Obsolescent|Deleted feature used|(?:[\w]+ )?Error|Fatal|Panic)(\(\w+\))?: (?<fname>[\S ]+), line (?<ln>\d+): (?<msg1>.+)$/gm;
491496

492497
default:
493498
vscode.window.showErrorMessage('Unsupported linter, change your linter.compiler option');

0 commit comments

Comments
 (0)