Skip to content

enabled valueFlowBailoutIncompleteVar in selfcheck - #5351

Draft
firewave wants to merge 2 commits into
cppcheck-opensource:mainfrom
firewave:self-incomplete
Draft

enabled valueFlowBailoutIncompleteVar in selfcheck#5351
firewave wants to merge 2 commits into
cppcheck-opensource:mainfrom
firewave:self-incomplete

Conversation

@firewave

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave

This comment was marked as outdated.

@firewave

This comment was marked as resolved.

@firewave
firewaveforce-pushed the self-incomplete branch 5 times, most recently from 5f616aa to c2299fbCompareSeptember 18, 2023 10:18
@chrchr-github

Copy link
Copy Markdown
Collaborator

Seems like the remaining bailouts are cfg-related, except for that uninstantiated(?) template parameter.

@firewave

This comment was marked as duplicate.

@chrchr-github

chrchr-github commented Sep 19, 2023

Copy link
Copy Markdown
Collaborator
gui/codeeditorstyle.cpp:125:40: debug: valueflow.cpp:10106:valueFlowConditionExpressions bailout: Skipping function due to incomplete variable widgetFGColor [valueFlowBailoutIncompleteVar]
QVariant(defaultStyleLight.widgetFGColor)).value<QColor>();
^

https://trac.cppcheck.net/ticket/10045#comment:30https://trac.cppcheck.net/ticket/12573

@firewave
firewaveforce-pushed the self-incomplete branch 3 times, most recently from a780558 to adae25dCompareSeptember 25, 2023 23:29
@firewave
firewaveforce-pushed the self-incomplete branch 2 times, most recently from b058082 to 1d25aa6CompareOctober 26, 2023 09:11
@firewave

This comment was marked as resolved.

@firewave

firewave commented Mar 22, 2024

Copy link
Copy Markdown
CollaboratorAuthor
lib/astutils.cpp:343:90: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable intvalue [valueFlowBailoutIncompleteVar]
if (!tok->varId() && tok->hasKnownIntValue() && std::to_string(tok->values().front().intvalue) == rhs)
^

https://trac.cppcheck.net/ticket/12539.

@firewave

This comment was marked as resolved.

@firewave

This comment was marked as resolved.

@firewave

firewave commented Apr 3, 2024

Copy link
Copy Markdown
CollaboratorAuthor

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

Seems like the remaining bailouts are cfg-related, except for that uninstantiated(?) template parameter.

The most common ones actually require special handling and cannot simply be solved via the configuration. I filed tickets for most of them as they are also the top results in daca.

@firewave

This comment was marked as resolved.

@firewave
firewaveforce-pushed the self-incomplete branch 4 times, most recently from a0fac37 to 346c833CompareApril 6, 2024 10:03
@firewave
firewaveforce-pushed the self-incomplete branch 2 times, most recently from c09ec05 to b589377CompareJune 20, 2024 22:21
@firewave
firewaveforce-pushed the self-incomplete branch 2 times, most recently from fa7718a to a98ed76CompareFebruary 6, 2025 13:23
@sonarqubecloud

Copy link
Copy Markdown

Sign up for freeto 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

@firewave@chrchr-github