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 — Fix #14119 FN constVariablePointer for variable declared in for loop / partial fix for #14148 by chrchr-github · Pull Request #7807 · cppcheck-opensource/cppcheck

Conversation

@chrchr-github

@chrchr-github chrchr-github commented Sep 8, 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
web-flow added 2 commits September 8, 2025 20:09
@web-flow
Fix #14119 FN constVariablePointer for variable declared in for loop
753501c
@web-flow
const
94c2aa3
@firewave

firewave commented Sep 8, 2025

Copy link
Copy Markdown
Collaborator

simplecpp.cpp changes must be merges from upstream. The changes are included in cppcheck-opensource/simplecpp#421. I will see if we can get that PR merged without the additional fixes.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@web-flow
const
70c7ca6
firewave
firewave reviewed Sep 8, 2025
View reviewed changes
Comment thread lib/token.cpp Resolved

Uh oh!

There was an error while loading. Please reload this page.
@web-flow
Partial fix for #14148
f833907
@chrchr-github chrchr-github mentioned this pull request Sep 23, 2025
clang-tidy.yml: updated to Clang 21 cppcheck-opensource/simplecpp#421
Merged
@chrchr-github chrchr-github changed the title Fix #14119 FN constVariablePointer for variable declared in for loop Fix #14119 FN constVariablePointer for variable declared in for loop / partial fix for #14148 Sep 24, 2025
@chrchr-github

chrchr-github commented Sep 29, 2025

Copy link
Copy Markdown
Collaborator Author

Now we need to tag a new simplecpp version and merge it...
All reactions

Sorry, something went wrong.

Uh oh!

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

firewave commented Sep 29, 2025 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown
Collaborator

Now we need to tag a new simplecpp version and merge it...

We still need to merge the MinGW stuff to make sure that is working. I will already prepare a draft though as we need Cppcheck side changes.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
chrchr-github added 2 commits October 2, 2025 11:22
@chrchr-github
Merge branch 'main' into chr_14119
33822ca
@chrchr-github
Update testother.cpp
83407ef
@danmar

danmar commented Oct 3, 2025

Copy link
Copy Markdown
Collaborator

Now we need to tag a new simplecpp version and merge it...

I want to tag and merge but there are open PRs i.e. about the search paths that I feel deserve to get in in our next release..
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
chrchr-github added 2 commits October 3, 2025 10:11
@chrchr-github
Update simplecpp.cpp
49490e3
@chrchr-github
Merge branch 'danmar:main' into chr_14119
29a493d
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 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.
@chrchr-github
chrchr-github merged commit 9ac99f0 into cppcheck-opensource:main Oct 7, 2025
@chrchr-github
chrchr-github deleted the chr_14119 branch October 7, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@firewave firewave firewave left review comments

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.

4 participants

@chrchr-github @firewave @danmar @web-flow
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.