Skip to content

C++: Upgrade cpp/comparison-with-wider-type to high precision#21634

Merged
geoffw0 merged 4 commits intogithub:mainfrom
geoffw0:compwidertype2
Apr 10, 2026
Merged

C++: Upgrade cpp/comparison-with-wider-type to high precision#21634
geoffw0 merged 4 commits intogithub:mainfrom
geoffw0:compwidertype2

Conversation

@geoffw0
Copy link
Copy Markdown
Contributor

@geoffw0 geoffw0 commented Apr 2, 2026

Upgrade cpp/comparison-with-wider-type to @precision high (following multiple rounds of testing).

@geoffw0
Copy link
Copy Markdown
Contributor Author

geoffw0 commented Apr 10, 2026

DCA, QA LGTM. Bringing out of draft.

@geoffw0 geoffw0 marked this pull request as ready for review April 10, 2026 12:58
@geoffw0 geoffw0 requested a review from a team as a code owner April 10, 2026 12:58
Copilot AI review requested due to automatic review settings April 10, 2026 12:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR upgrades the C++ query cpp/comparison-with-wider-type to high precision, which makes it eligible to run in the default C++ code scanning suite, and documents the change.

Changes:

  • Updated the query metadata for cpp/comparison-with-wider-type from @precision medium to @precision high.
  • Added a change note announcing the precision upgrade and default-suite inclusion impact.
  • Updated the expected contents of the C++ code scanning query suite to include the query.
Show a summary per file
File Description
cpp/ql/src/Security/CWE/CWE-190/ComparisonWithWiderType.ql Changes query metadata to @precision high.
cpp/ql/src/change-notes/2026-04-02-comparison-with-wider-type.md Adds release note for the precision upgrade and suite impact.
cpp/ql/integration-tests/query-suite/cpp-code-scanning.qls.expected Updates expected default code scanning suite contents to include the query.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

@geoffw0 geoffw0 merged commit ae85ada into github:main Apr 10, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants