◐ Shell
reader mode source ↗
Skip to content

ValueFlow: pass ErrorLogger by reference into ValueFlow::setValues() / removed need for LifetimeStore::Context#5299

Merged
firewave merged 9 commits into
cppcheck-opensource:mainfrom
firewave:ptr-ref-v
Apr 4, 2024
Merged

ValueFlow: pass ErrorLogger by reference into ValueFlow::setValues() / removed need for LifetimeStore::Context#5299
firewave merged 9 commits into
cppcheck-opensource:mainfrom
firewave:ptr-ref-v

Conversation

@firewave

@firewave firewave commented Aug 7, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

@danmar danmar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hide comment

I am fine with that.

@firewave firewave marked this pull request as draft August 7, 2023 18:39
@firewave

firewave commented Aug 7, 2023

Copy link
Copy Markdown
Collaborator Author

Needs some reworking around givenACodeSampleToTokenize first.

@firewave firewave force-pushed the ptr-ref-v branch 4 times, most recently from e7bc1ff to ed3cb49 Compare March 9, 2024 16:42
@firewave firewave changed the title ValueFlow: pass ErrorLogger by reference into ValueFlow::setValues() Mar 9, 2024
@firewave firewave force-pushed the ptr-ref-v branch 4 times, most recently from d0d4904 to 38c8092 Compare March 9, 2024 23:15
@firewave firewave force-pushed the ptr-ref-v branch 4 times, most recently from 111292c to 53dd484 Compare March 11, 2024 10:08
@firewave firewave requested a review from danmar March 11, 2024 10:19
@firewave firewave force-pushed the ptr-ref-v branch 2 times, most recently from 8eac401 to 39369b4 Compare March 11, 2024 15:02
@firewave firewave marked this pull request as ready for review March 12, 2024 00:15
@firewave firewave merged commit 228f9cd into cppcheck-opensource:main Apr 4, 2024
@firewave firewave deleted the ptr-ref-v branch April 4, 2024 13:39
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.

3 participants