Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cppcheck - various fixes #8622

Merged
merged 1 commit into from
Nov 26, 2024
Merged

cppcheck - various fixes #8622

merged 1 commit into from
Nov 26, 2024

Conversation

stsoe
Copy link
Collaborator

@stsoe stsoe commented Nov 25, 2024

Problem solved by the commit

Fix few cppcheck warnings

How problem was solved, alternative solutions (if any) and why they were rejected

Add nullptr check; not that this will ever trigger, but sure, cppcheck is correct about missing nullptr check.

Use override on virtual function overrides, but leave virtual if the override makes the function pure virtual for additional derived classes.

Use explicit to avoid converting constructors, but allow converting constructors for classes that act more or less as alias for type.

@stsoe stsoe requested review from rozumx and uday610 as code owners November 25, 2024 23:20
Add nullptr check; not that this will ever trigger, but sure, cppcheck
is correct about missing nullptr check.

Use override on virtual function overrides, but leave virtual if the
override makes the function pure virtual for additional derived classes.

Use explicit to avoid converting constructors, but allow converting
constructors for classes that act more or less as alias for type.

Signed-off-by: Soren Soe <[email protected]>
@stsoe stsoe merged commit f093392 into Xilinx:master Nov 26, 2024
20 checks passed
@stsoe stsoe deleted the cppcheck branch November 26, 2024 02:59
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.

1 participant