GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fixed #14331 - suppressions with line number not reported as unmatched by firewave · Pull Request #8035 · cppcheck-opensource/cppcheck

Conversation

@firewave

@firewave firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@firewave
firewave force-pushed the unmatch-line branch from f6da85f to 39c5aad Compare December 15, 2025 01:41
@firewave firewave changed the title fixed # - suppressions with line number not reported as unmatched fixed #14331 - suppressions with line number not reported as unmatched Dec 15, 2025
@firewave
firewave force-pushed the unmatch-line branch 2 times, most recently from ae7c0c4 to 653e0e6 Compare December 15, 2025 12:03
@firewave
firewave marked this pull request as ready for review December 15, 2025 12:03
@firewave
firewave force-pushed the unmatch-line branch from 653e0e6 to 381a437 Compare December 15, 2025 12:36
@firewave

firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator Author

The cfg test failures are caused by missing configurations.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave

firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator Author

The Python test is passing locally for me - needs more looking into.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave

firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator Author

The cfg test failures are caused by missing configurations.

See at least https://trac.cppcheck.net/ticket/1059#comment:11 among possibly other tickets.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave

firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator Author

The Python test is passing locally for me - needs more looking into.

Ah, they fail only with -j2.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave
firewave marked this pull request as draft December 15, 2025 12:45
@firewave

firewave commented Dec 15, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator Author

The cfg test failures are caused by missing configurations.

See at least https://trac.cppcheck.net/ticket/1059#comment:11 among possibly other tickets.

Actually this is related to __cplusplus already being defined (always). When macros are flagged as built-in (see cppcheck-opensource/simplecpp#547) we might be able to add some handling for this.

I filed https://trac.cppcheck.net/ticket/14333 about it.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave

firewave commented Dec 15, 2025

Copy link
Copy Markdown
Collaborator Author

This needs more looking into regarding inline suppressions in inactive code.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
firewave added 2 commits December 29, 2025 21:22
@firewave
fixed #14331 - suppressions with line number not reported as unmatched
028ebec
@firewave
mitigated unmatchedSuppression warnings in cfg tests [skip ci]
2ee21d7
@firewave
firewave force-pushed the unmatch-line branch from 381a437 to 2ee21d7 Compare December 29, 2025 20:22
@sonarqubecloud

sonarqubecloud Bot commented Dec 29, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@firewave

firewave commented Dec 29, 2025

Copy link
Copy Markdown
Collaborator Author

The Python test is passing locally for me - needs more looking into.

Ah, they fail only with -j2.

This is an issue unrelated to my changes. I filed https://trac.cppcheck.net/ticket/14350 about it.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@firewave
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.