Skip to content

some minor QtCharts usage cleanups#4890

Merged
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:qt-x
Mar 13, 2023
Merged

some minor QtCharts usage cleanups#4890
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:qt-x

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Mar 13, 2023

The methods were not reported as functionStatic because the base class QDialog is unknown and they might override base class methods without using override or virtual so we bail out.

@danmar danmar merged commit 0b8af4f into cppcheck-opensource:main Mar 13, 2023
@firewave firewave deleted the qt-x branch March 13, 2023 14:53
danmar pushed a commit that referenced this pull request Mar 23, 2023
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.

2 participants