Skip to content

refs #14265: some simplecpp errors might lack column information - #7965

Merged
firewave merged 3 commits into
cppcheck-opensource:mainfrom
ludviggunne:14265
Nov 17, 2025
Merged

refs #14265: some simplecpp errors might lack column information#7965
firewave merged 3 commits into
cppcheck-opensource:mainfrom
ludviggunne:14265

Conversation

@ludviggunne

@ludviggunneludviggunne commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@ludviggunne

ludviggunne commented Nov 17, 2025•
edited
Loading

Copy link
Copy Markdown
CollaboratorAuthor

@firewave just saw that you were assigned this in trac... if you already have a fix I can close this.

@firewavefirewave changed the title fix #14265: some simplecpp errors might lack column informationrefs #14265: some simplecpp errors might lack column informationNov 17, 2025
@firewave

firewave commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

@firewave just saw that you were assigned this in trac... if you already have a fix I can close this.

Yes, I am quite deep into this right now and have so many changes that I need to take the usual very incremental steps to a grip on this.

The change is fine though but I think it is not fixing all cases as some are only added in #7960 (which I was about to fix/rebase). So I changed the title.

@firewave

firewave commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

Also it looks like a column is wrong but that is out of the scope of this PR (will file a ticket later - there's actually a couple of other related tickets I have not filed yet).

@sonarqubecloud

sonarqubecloudBot commented Nov 17, 2025

Copy link
Copy Markdown

Quality Gate PassedQuality 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

@firewave
firewave merged commit bb78bd8 into cppcheck-opensource:mainNov 17, 2025
55 checks passed
@ludviggunne
ludviggunne deleted the 14265 branch November 17, 2025 16:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@firewavefirewavefirewave approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ludviggunne@firewave