Skip to content

Implement MISRA-C++23 Preprocesser package rules 19-0-4, 19-1-1, and 19-2-1 #893

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Implement MISRA-C++23 Preprocesser package rules 19-0-4, 19-1-1, and 19-2-1 #893

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9830cc0
Implement MISRA-C++23 Preprocesser package rules 19-0-4, 19-1-1, and …
MichaelRFairhurst Apr 29, 2025
2095d1d
Fix Preprocessor.json, for defined in if directive
MichaelRFairhurst Apr 29, 2025
ef9d96b
Fix package errors
MichaelRFairhurst Apr 29, 2025
a49c0d1
s/maintanability/maintainability
MichaelRFairhurst Apr 29, 2025
ea173e5
Regenerate query metadata
MichaelRFairhurst Apr 29, 2025
a31e047
Format additional files
MichaelRFairhurst Apr 29, 2025
b47ec45
Merge branch 'main' into michaelrfairhurst/implement-package-preproce…
lcartey Jun 9, 2025
39ef003
Update to use features from codeql-qtil
MichaelRFairhurst Aug 24, 2025
5da3a00
Merge remote-tracking branch 'origin/main' into michaelrfairhurst/imp…
MichaelRFairhurst Aug 24, 2025
eb5453b
Update to 0.0.3 which has fixes for codeql/util breaking change.
MichaelRFairhurst Aug 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix Preprocessor.json, for defined in if directive
  • Loading branch information
@MichaelRFairhurst
MichaelRFairhurst committed Apr 29, 2025
commit 2095d1d2dbf9608c2e6d4117029292bbbb63515f
2 changes: 1 addition & 1 deletion cpp/misra/src/rules/RULE-19-1-1/DefinedOperatorExpandedInIfDirective.ql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import codingstandards.cpp.misra

from PreprocessorIf ifDirective, MacroInvocation mi
where
not isExcluded(ifDirective, PreprocessorPackage::macroDefinitionContainsDefinedOperatorQuery()) and
not isExcluded(ifDirective, PreprocessorPackage::definedOperatorExpandedInIfDirectiveQuery()) and
ifDirective.getLocation().subsumes(mi.getLocation()) and
mi.getMacro().getBody().regexpMatch(".*defined.*")
select ifDirective,
Expand Down
40 changes: 0 additions & 40 deletions rule_packages/cpp/Preprocessor.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,46 +98,6 @@
}
],
"title": "The #include directive shall be followed by either a <filename> or \"filename\" sequence"
},
"RULE-19-3-4": {
"properties": {
"enforcement": "decidable",
"obligation": "required"
},
"queries": [
{
"description": "",
"kind": "problem",
"name": "Parentheses shall be used to ensure macro arguments are expanded appropriately",
"precision": "very-high",
"severity": "error",
"short_name": "ParenthesesNotUsedToEnsureMacroArgumentsAreExpandedAppropriately",
"tags": [
"scope/single-translation-unit"
]
}
],
"title": "Parentheses shall be used to ensure macro arguments are expanded appropriately"
},
"RULE-19-6-1": {
"properties": {
"enforcement": "decidable",
"obligation": "advisory"
},
"queries": [
{
"description": "",
"kind": "problem",
"name": "The #pragma directive and the _Pragma operator should not be used",
"precision": "very-high",
"severity": "error",
"short_name": "PragmaDirectiveAndThePragmaOperatorShouldNotBeUsed",
"tags": [
"scope/single-translation-unit"
]
}
],
"title": "The #pragma directive and the _Pragma operator should not be used"
}
}
}
Loading