Skip to content

Conversation

@ludviggunne
Copy link
Collaborator

No description provided.

@ludviggunne
Copy link
Collaborator Author

ludviggunne commented Nov 17, 2025

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

@firewave firewave changed the title fix #14265: some simplecpp errors might lack column information refs #14265: some simplecpp errors might lack column information Nov 17, 2025
@firewave
Copy link
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
Copy link
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
Copy link

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants