Skip to content

C++: Add static analysis for generated C++ bindings#128

Merged
bact merged 5 commits into
JPEWdev:mainfrom
bact:add-cpp-cppcheck
Jul 8, 2026
Merged

C++: Add static analysis for generated C++ bindings#128
bact merged 5 commits into
JPEWdev:mainfrom
bact:add-cpp-cppcheck

Conversation

@bact

@bact bact commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Use cppcheck for static analysis.

See #120

Got these warnings/errors:

objectset.cpp:43:21: performance: Searching before insertion is not necessary. [stlFindInsert]

objectset.cpp:247:8: error: Using object that is a temporary. [danglingTemporaryLifetime]

Use cppcheck

Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
@bact bact added the tests Tests, test spec, test data label Jul 4, 2026
@bact bact marked this pull request as draft July 4, 2026 20:08
@bact bact changed the title C++: Add static analysis for generated C++ code C++: Add static analysis for generated C++ bindings Jul 4, 2026
bact added 2 commits July 6, 2026 17:53
Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

bact added 2 commits July 8, 2026 00:19
Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
Use clang-format

Signed-off-by: Arthit Suriyawongkul <arthit@gmail.com>
@bact bact marked this pull request as ready for review July 8, 2026 00:25
@JPEWdev JPEWdev self-requested a review July 8, 2026 14:49
@bact bact merged commit 3f21976 into JPEWdev:main Jul 8, 2026
9 checks passed
@bact bact deleted the add-cpp-cppcheck branch July 8, 2026 15:19
@bact bact added this to the 1.2.0 milestone Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests, test spec, test data

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants