seqan / hibf

HIBF and IBF
https://docs.seqan.de/hibf
Other
4 stars 2 forks source link

[MISC] Fix some clang-tidy warnings #145

Closed eseiler closed 1 year ago

eseiler commented 1 year ago

Mainly move ctor/assignment not begin noexcept and move of const variables (which has no effect).

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hibf ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 19, 2023 5:13pm
codecov[bot] commented 1 year ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (7b489d6) 99.77% compared to head (042b80d) 99.70%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #145 +/- ## ========================================== - Coverage 99.77% 99.70% -0.08% ========================================== Files 41 41 Lines 1351 1355 +4 ========================================== + Hits 1348 1351 +3 - Misses 3 4 +1 ``` | [Files](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan) | Coverage Δ | | |---|---|---| | [include/hibf/misc/insert\_iterator.hpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-aW5jbHVkZS9oaWJmL21pc2MvaW5zZXJ0X2l0ZXJhdG9yLmhwcA==) | `100.00% <100.00%> (ø)` | | | [src/hierarchical\_interleaved\_bloom\_filter.cpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-c3JjL2hpZXJhcmNoaWNhbF9pbnRlcmxlYXZlZF9ibG9vbV9maWx0ZXIuY3Bw) | `100.00% <ø> (ø)` | | | [src/interleaved\_bloom\_filter.cpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-c3JjL2ludGVybGVhdmVkX2Jsb29tX2ZpbHRlci5jcHA=) | `100.00% <ø> (ø)` | | | [src/layout/graph.cpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-c3JjL2xheW91dC9ncmFwaC5jcHA=) | `100.00% <100.00%> (ø)` | | | [src/layout/hierarchical\_binning.cpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-c3JjL2xheW91dC9oaWVyYXJjaGljYWxfYmlubmluZy5jcHA=) | `98.81% <100.00%> (+<0.01%)` | :arrow_up: | | [src/layout/simple\_binning.cpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-c3JjL2xheW91dC9zaW1wbGVfYmlubmluZy5jcHA=) | `100.00% <100.00%> (ø)` | | | [include/hibf/interleaved\_bloom\_filter.hpp](https://app.codecov.io/gh/seqan/hibf/pull/145?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=seqan#diff-aW5jbHVkZS9oaWJmL2ludGVybGVhdmVkX2Jsb29tX2ZpbHRlci5ocHA=) | `99.19% <88.88%> (-0.81%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.