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

added simplecpp source to selfcheck #5432

Merged
merged 3 commits into from
Oct 26, 2023

Conversation

firewave
Copy link
Collaborator

We do not have Cppcheck available in the simplecpp CI so we should check it here.

@firewave
Copy link
Collaborator Author

firewave commented Sep 11, 2023

The fixes for the warnings have been submitted upstream in danmar/simplecpp#315.

Still need to reduce the symbolDatabaseWarning warnings.

@firewave
Copy link
Collaborator Author

I filed https://trac.cppcheck.net/ticket/11978 for one of the symbolDatabaseWarning.

@firewave firewave force-pushed the selfcheck-simplecpp branch 2 times, most recently from 7c0207a to 5fcdfc2 Compare September 13, 2023 10:03
@firewave
Copy link
Collaborator Author

Another symbolDatabaseWarning: https://trac.cppcheck.net/ticket/11986

@firewave
Copy link
Collaborator Author

Yet another symbolDatabaseWarning: https://trac.cppcheck.net/ticket/12008.

@firewave firewave force-pushed the selfcheck-simplecpp branch from 3da7dba to c5256a2 Compare October 25, 2023 13:01
@firewave firewave marked this pull request as ready for review October 25, 2023 13:31
@firewave firewave force-pushed the selfcheck-simplecpp branch 2 times, most recently from 88236cf to 2b0fdfd Compare October 25, 2023 19:07
@firewave firewave force-pushed the selfcheck-simplecpp branch from 2b0fdfd to c9c215b Compare October 25, 2023 23:10
@chrchr-github chrchr-github merged commit 5bebdf8 into danmar:main Oct 26, 2023
74 checks passed
@firewave firewave deleted the selfcheck-simplecpp branch October 26, 2023 09:10
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