stotko / stdgpu

stdgpu: Efficient STL-like Data Structures on the GPU
https://stotko.github.io/stdgpu/
Apache License 2.0
1.16k stars 83 forks source link

src,test: Enable and fix more clang-tidy warnings #327

Closed stotko closed 2 years ago

stotko commented 2 years ago

Recent versions of clang-tidy introduced further checks to improve the code quality. Although our coverage is already quite good, increase the number of checks and suppress the aliases to existing checks.

In the future, we may even maximize the coverage by unconditionally enabling all checks and providing a single list of ignored warnings.

codecov[bot] commented 2 years ago

Codecov Report

Base: 97.14% // Head: 97.14% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (f670098) compared to base (83cefd4). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #327 +/- ## ======================================= Coverage 97.14% 97.14% ======================================= Files 32 32 Lines 2346 2347 +1 ======================================= + Hits 2279 2280 +1 Misses 67 67 ``` | [Impacted Files](https://codecov.io/gh/stotko/stdgpu/pull/327?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko) | Coverage Δ | | |---|---|---| | [src/stdgpu/bitset.cuh](https://codecov.io/gh/stotko/stdgpu/pull/327/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9iaXRzZXQuY3Vo) | `100.00% <ø> (ø)` | | | [src/stdgpu/impl/bitset\_detail.cuh](https://codecov.io/gh/stotko/stdgpu/pull/327/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9pbXBsL2JpdHNldF9kZXRhaWwuY3Vo) | `99.21% <ø> (ø)` | | | [src/stdgpu/impl/limits\_detail.h](https://codecov.io/gh/stotko/stdgpu/pull/327/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9pbXBsL2xpbWl0c19kZXRhaWwuaA==) | `100.00% <ø> (ø)` | | | [src/stdgpu/impl/atomic\_detail.cuh](https://codecov.io/gh/stotko/stdgpu/pull/327/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9pbXBsL2F0b21pY19kZXRhaWwuY3Vo) | `99.59% <100.00%> (ø)` | | | [src/stdgpu/openmp/impl/memory.cpp](https://codecov.io/gh/stotko/stdgpu/pull/327/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9vcGVubXAvaW1wbC9tZW1vcnkuY3Bw) | `65.21% <100.00%> (+1.58%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.