◐ Shell
reader mode source ↗
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

@firewave firewave commented Aug 19, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave

This comment was marked as outdated.

Sign in to view
@firewave

This comment was marked as resolved.

Sign in to view
@firewave firewave force-pushed the self-incomplete branch 5 times, most recently from 5f616aa to c2299fb Compare September 18, 2023 10:18
@chrchr-github

chrchr-github commented Sep 19, 2023

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.

Sign in to view
@chrchr-github

chrchr-github commented Sep 19, 2023 •
edited by firewave
Loading

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:30 https://trac.cppcheck.net/ticket/12573

@firewave firewave force-pushed the self-incomplete branch 3 times, most recently from a780558 to adae25d Compare September 25, 2023 23:29
@firewave firewave force-pushed the self-incomplete branch 2 times, most recently from b058082 to 1d25aa6 Compare October 26, 2023 09:11
@firewave

This comment was marked as resolved.

Sign in to view
@firewave

firewave commented Mar 22, 2024 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author 
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.

Sign in to view
@firewave

This comment was marked as resolved.

Sign in to view
@firewave

firewave commented Apr 3, 2024 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

I added a few more tickets:
https://trac.cppcheck.net/ticket/10949 - npos
https://trac.cppcheck.net/ticket/12577 - endl
https://trac.cppcheck.net/ticket/12578 - cout/cerr
https://trac.cppcheck.net/ticket/12580 - errno
https://trac.cppcheck.net/ticket/12581 - __func__
https://trac.cppcheck.net/ticket/13309

@firewave

firewave commented Apr 3, 2024

Copy link
Copy Markdown
Collaborator Author

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.

Sign in to view
@firewave firewave force-pushed the self-incomplete branch 4 times, most recently from a0fac37 to 346c833 Compare April 6, 2024 10:03
@firewave firewave force-pushed the self-incomplete branch 2 times, most recently from c09ec05 to b589377 Compare June 20, 2024 22:21
@firewave firewave force-pushed the self-incomplete branch 2 times, most recently from fa7718a to a98ed76 Compare February 6, 2025 13:23
@sonarqubecloud

sonarqubecloud Bot commented Jan 3, 2026

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

@cppcheck-opensource cppcheck-opensource deleted a comment from sonarqubecloud Bot May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@firewave @chrchr-github