Skip to content

Fix FP leakNoVarFunctionCall with Qt object#4272

Merged
orbitcowboy merged 12 commits intocppcheck-opensource:mainfrom
chrchr-github:chr_FixQtLeak
Jul 13, 2022
Merged

Fix FP leakNoVarFunctionCall with Qt object#4272
orbitcowboy merged 12 commits intocppcheck-opensource:mainfrom
chrchr-github:chr_FixQtLeak

Conversation

@chrchr-github
Copy link
Copy Markdown
Collaborator

Although connect() was missing leak-ignore, the FP came up since windows.cfg also has a connect().

The warning for HelpDialog looks like a true positive to me.

@chrchr-github
Copy link
Copy Markdown
Collaborator Author

Seems like <QShortcut> is not available for some reason.

@firewave
Copy link
Copy Markdown
Collaborator

firewave commented Jul 13, 2022

Seems like <QShortcut> is not available for some reason.

It requires the Gui component and runtests.sh only seems to utilize Core (i.e. Qt5Core).

@chrchr-github
Copy link
Copy Markdown
Collaborator Author

Seems like <QShortcut> is not available for some reason.

It requires the Gui component and runtests.sh only seems to utilize Core (i.e. Qt5Core).

Yeah, I'm trying to work around that now...

@orbitcowboy orbitcowboy merged commit 2c7d986 into cppcheck-opensource:main Jul 13, 2022
@chrchr-github chrchr-github deleted the chr_FixQtLeak branch October 22, 2022 09:47
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