Skip to content

Commit

Permalink
No longer omit inline unmatchedSuppression findings - fixes johntha…
Browse files Browse the repository at this point in the history
  • Loading branch information
firewave committed Apr 13, 2023
1 parent 7a2ff9a commit e8c3f7a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,7 @@ Deployment.
- Report execution errors as global inspection errors.
- Display `Cppcheck Path` configuration errors as global inspection errors instead of using a (hard to spot) status bar message.
- Display global inspection error and omit the option if the configured `MISRA Addon JSON` does not exist.
- No longer omit inline `unmatchedSuppression` findings.

### 1.6.2 - 2022-01-25

Expand Down
1 change: 1 addition & 0 deletions resources/META-INF/plugin.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
- Report execution errors as global inspection errors.
- Display `Cppcheck Path` configuration errors as global inspection errors instead of using a (hard to spot) status bar message.
- Display global inspection error and omit the option if the configured `MISRA Addon JSON` does not exist.
- No longer omit inline `unmatchedSuppression` findings.
]]>
</change-notes>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ public static List<ProblemDescriptor> parseOutput(@NotNull final PsiFile psiFile
}

// we are never interested in these
if (id.equals("unmatchedSuppression") || id.equals("purgedConfiguration")) {
if (id.equals("purgedConfiguration")) {
continue;
}

Expand Down Expand Up @@ -178,6 +178,7 @@ public static List<ProblemDescriptor> parseOutput(@NotNull final PsiFile psiFile
// If a file #include's header files, Cppcheck will also run on the header files and print
// any errors. These errors don't apply to the current file and should not be drawn. They can
// be distinguished by checking the file name.
// Will also exclude non-inline 'unmatchedSuppression' findings.
if (!fileName.equals(sourceFileName)) {
continue;
}
Expand Down

0 comments on commit e8c3f7a

Please sign in to comment.